swagcov 0.2.1 → 0.2.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 326ccddda2c92236e815bc06b7a29f64b5b4a1d59a796f000c5c424b91b1f7cd
-  data.tar.gz: 557e962da1fb3232dd7ed29e05597a88771c9ce1f34fd35ec39b5140c901ed65
+  metadata.gz: 820849bdcc13ddb85e1ff404bcc881dbe8f8154090bb6111d0afbdbdd286ce4f
+  data.tar.gz: 90de5bb3a335673d58524344dd871182e06c1eb3185192a92026be5be0ce372e
 SHA512:
-  metadata.gz: f6c067ae52cb5e3609e0cab96e08b3e016438bbf7ed04281bd2ce024e2af0bd47e601aa3ba85dca61b1c973dca495431b635be1b24d8ef7dbf8fbcf7f5218d90
-  data.tar.gz: 8caf8633ab01634c0fa42a38edc44f07127f4bd984792144562b7274ba46f31a9e0af52828ce8221f678c61866e9b7c6ccbb5ffe14935c9b2b0e9f0668c7ef1b
+  metadata.gz: b49673a8bfca29bb64ece42acc8c1a85f34339b4aa643036c72f4c1849e6e061a933c6471df2502da4e5ba4676c4f75cec559518c87242c7a3a31d6c0e817000
+  data.tar.gz: 1b60f5d474d4be3c5c51a6180cb0343b380bca69a4fa99fe0c55063e7037a4f1134d3cbd284a411aa2a9b6ef257898df55facdf99b8827fe2fde3c069c8222be

data/README.md

@@ -1,5 +1,10 @@
 # Swagcov
-See coverage report for openapi docs for Rails Routes.
+[![Gem Version](https://img.shields.io/gem/v/swagcov)](https://rubygems.org/gems/swagcov)
+![Gem Downloads](https://img.shields.io/gem/dt/swagcov)
+[![Ruby Style Guide](https://img.shields.io/badge/code_style-rubocop-brightgreen.svg)](https://github.com/rubocop-hq/rubocop)
+[![GitHub License](https://img.shields.io/github/license/smridge/swagcov.svg)](https://github.com/smridge/swagcov/blob/main/LICENSE)
+
+See OpenAPI documentation coverage report for Rails Routes.
 
 ## Installation
 Add this line to your application's Gemfile:
@@ -34,7 +39,6 @@ Create a `.swagcov.yml` in root of your Rails application.
     paths:
       ignore:
         - /v1/foobar/:token
-        - /sidekiq
   ```
 
 - Example `.swagcov.yml` Config File:
@@ -49,7 +53,6 @@ Create a `.swagcov.yml` in root of your Rails application.
         - ^/v1
       ignore:
         - /v1/foobar/:token
-        - /sidekiq
   ```
 
 Execute:
@@ -57,6 +60,44 @@ Execute:
 bundle exec rake swagcov
 ```
 
+## Example configurations and output from running `bundle exec rake swagcov` from the root of your Rails Application:
+- All Routes (minimal configuration):
+  ```yml
+  docs:
+    paths:
+      - swagger.yaml
+  ```
+  <img src="https://raw.githubusercontent.com/smridge/swagcov/main/images/all-endpoints.png">
+
+
+- With `only` endpoint configuration:
+  ```yml
+  docs:
+    paths:
+      - swagger.yaml
+
+  routes:
+    paths:
+      only:
+        - ^/v2
+  ```
+  <img src="https://raw.githubusercontent.com/smridge/swagcov/main/images/only-endpoints.png">
+
+- With `ignore` and `only` endpoint configurations:
+  ```yml
+  docs:
+    paths:
+      - swagger/v1/swagger.yaml
+
+  routes:
+    paths:
+      only:
+        - ^/v2
+      ignore:
+        - /v2/users
+  ```
+  <img src="https://raw.githubusercontent.com/smridge/swagcov/main/images/ignore-and-only-endpoints.png">
+
 ## Development
 ```shell
 git clone git@github.com:smridge/swagcov.git
@@ -72,18 +113,34 @@ gem "swagcov", path: "../swagcov"
 bundle
 ```
 
+Run Tests
+```
+bundle exec rspec spec --exclude-pattern spec/sandbox_5_2/**/*_spec.rb
+```
+
+### Test via Sandbox Application
+For Rails 5
+- `cd spec/sandbox_5_2/`
+- Run tests: `bundle exec rspec spec`
+- Run `bundle exec rake swagcov`
+  - This will run against any changes made to your branch.
+
 ## Publish (internal)
 > Note: Publishing a new version of this gem is only meant for maintainers.
 - Ensure you have access to publish on [rubygems](https://rubygems.org/gems/swagcov).
 - Update [CHANGELOG](https://github.com/smridge/swagcov/blob/main/CHANGELOG.md).
 - Update [`VERSION`](https://github.com/smridge/swagcov/blob/main/lib/swagcov/version.rb).
+- Commit changes to `main` branch locally.
 - Run: `rake release`
   - This command builds the gem, creates a tag and publishes to rubygems, see [bundler docs](https://bundler.io/guides/creating_gem.html#releasing-the-gem).
 
 ## TODO
 - Add specs
 - Test against different rails versions
+- Create Sandbox App for Rails 6
 - Add autogeneration of ignore paths
+- Add `CONTRIBUTING.md`
+- Add GitHub Actions for specs/linting
 
 ## Credit
 To [@lonelyelk](https://github.com/lonelyelk) for initial development!

data/lib/swagcov/coverage.rb

@@ -2,29 +2,50 @@
 
 module Swagcov
   class Coverage
-    def initialize
+    attr_reader :total, :covered, :ignored, :routes_not_covered, :routes_covered, :routes_ignored
+
+    def initialize dotfile: Swagcov::Dotfile.new, routes: ::Rails.application.routes.routes
       @total = 0
       @covered = 0
       @ignored = 0
       @routes_not_covered = []
       @routes_covered = []
       @routes_ignored = []
+      @dotfile = dotfile
+      @routes = routes
     end
 
     def report
-      Rails.application.routes.routes.each do |route|
+      collect_coverage
+      routes_output(@routes_covered, "green")
+      routes_output(@routes_ignored, "yellow")
+      routes_output(@routes_not_covered, "red")
+
+      final_output
+
+      exit @total - @covered
+    end
+
+    private
+
+    attr_reader :dotfile
+
+    def collect_coverage
+      @routes.each do |route|
         path = route.path.spec.to_s.sub(/\(\.:format\)$/, "")
 
-        if ignore_path?(path)
+        next if third_party_route?(route, path)
+
+        if dotfile.ignore_path?(path)
           @ignored += 1
           @routes_ignored << { verb: route.verb, path: path, status: "ignored" }
           next
         end
 
-        next if only_path_mismatch?(path)
+        next if dotfile.only_path_mismatch?(path)
 
         @total += 1
-        regex = Regexp.new("#{path.gsub(%r{:[^/]+}, '\\{[^/]+\\}')}(\\.[^/]+)?$")
+        regex = Regexp.new("^#{path.gsub(%r{:[^/]+}, '\\{[^/]+\\}')}(\\.[^/]+)?$")
         matching_keys = docs_paths.keys.select { |k| regex.match?(k) }
 
         if (doc = docs_paths.dig(matching_keys.first, route.verb.downcase))
@@ -34,57 +55,33 @@ module Swagcov
           @routes_not_covered << { verb: route.verb, path: path, status: "none" }
         end
       end
-
-      routes_output(@routes_covered, "green")
-      routes_output(@routes_ignored, "yellow")
-      routes_output(@routes_not_covered, "red")
-
-      final_output
-
-      exit @total - @covered
-    end
-
-    def dotfile
-      @dotfile ||= YAML.load_file(Rails.root.join(".swagcov.yml"))
     end
 
     def docs_paths
-      @docs_paths ||= Dir.glob(dotfile["docs"]["paths"]).reduce({}) do |acc, docspath|
+      @docs_paths ||= Dir.glob(dotfile.doc_paths).reduce({}) do |acc, docspath|
         acc.merge(YAML.load_file(docspath)["paths"])
       end
     end
 
-    private
-
-    def ignore_path? path
-      ignored_regex&.match?(path)
-    end
-
-    def ignored_regex
-      @ignored_regex ||= path_config_regex(dotfile.dig("routes", "paths", "ignore"))
-    end
-
-    def only_path_mismatch? path
-      only_regex && !only_regex.match?(path)
-    end
-
-    def only_regex
-      @only_regex ||= path_config_regex(dotfile.dig("routes", "paths", "only"))
-    end
-
-    def path_config_regex path_config
-      return unless path_config
+    def third_party_route? route, path
+      # https://github.com/rails/rails/blob/48f3c3e201b57a4832314b2c957a3b303e89bfea/actionpack/lib/action_dispatch/routing/inspector.rb#L105-L107
+      # Skips route paths like ["/rails/info/properties", "/rails/info", "/rails/mailers"]
+      route.internal ||
 
-      config = path_config.map { |path| path.first == "^" ? path : "#{path}$" }
+        # Skips routes like "/sidekiq"
+        route.verb.blank? ||
 
-      /#{config.join('|')}/
+        # Exclude routes that are part of the rails gem that you would not write documentation for
+        # https://github.com/rails/rails/tree/main/activestorage/app/controllers/active_storage
+        # https://github.com/rails/rails/tree/main/actionmailbox/app/controllers/action_mailbox
+        path.include?("/active_storage/") || path.include?("/action_mailbox/")
     end
 
     def routes_output routes, status_color
       routes.each do |route|
         $stdout.puts(
           format(
-            "%<verb>10s    %<path>-90s    %<status>s",
+            "%<verb>10s %<path>-90s %<status>s",
             { verb: route[:verb], path: route[:path], status: route[:status].send(status_color) }
           )
         )

data/lib/swagcov/dotfile.rb

@@ -0,0 +1,53 @@
+# frozen_string_literal: true
+
+module Swagcov
+  class BadConfigurationError < RuntimeError
+  end
+
+  class Dotfile
+    DEFAULT_CONFIG_FILE_NAME = ".swagcov.yml"
+
+    def initialize pathname: Rails.root.join(DEFAULT_CONFIG_FILE_NAME)
+      raise BadConfigurationError, "Missing config file (#{DEFAULT_CONFIG_FILE_NAME})" unless pathname.exist?
+
+      @dotfile = YAML.load_file(pathname)
+      raise BadConfigurationError, "Invalid config file (#{DEFAULT_CONFIG_FILE_NAME})" unless valid?
+    end
+
+    def ignore_path? path
+      ignored_regex&.match?(path)
+    end
+
+    def only_path_mismatch? path
+      only_regex && !only_regex.match?(path)
+    end
+
+    def doc_paths
+      dotfile.dig("docs", "paths")
+    end
+
+    private
+
+    attr_reader :dotfile
+
+    def ignored_regex
+      @ignored_regex ||= path_config_regex(dotfile.dig("routes", "paths", "ignore"))
+    end
+
+    def only_regex
+      @only_regex ||= path_config_regex(dotfile.dig("routes", "paths", "only"))
+    end
+
+    def path_config_regex path_config
+      return unless path_config
+
+      config = path_config.map { |path| path.first == "^" ? path : "^#{path}$" }
+
+      /#{config.join('|')}/
+    end
+
+    def valid?
+      dotfile && doc_paths
+    end
+  end
+end

data/lib/swagcov/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Swagcov
-  VERSION = "0.2.1"
+  VERSION = "0.2.5"
 end

data/lib/swagcov.rb

@@ -4,6 +4,7 @@ require "swagcov/version"
 
 if defined?(Rails)
   require "swagcov/railtie"
+  require "swagcov/dotfile"
   require "swagcov/coverage"
 end
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: swagcov
 version: !ruby/object:Gem::Version
-  version: 0.2.1
+  version: 0.2.5
 platform: ruby
 authors:
 - Sarah Ridge
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-04-21 00:00:00.000000000 Z
+date: 2021-09-14 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -24,6 +24,20 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '5'
+- !ruby/object:Gem::Dependency
+  name: pry-byebug
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: rspec
   requirement: !ruby/object:Gem::Requirement
@@ -66,6 +80,20 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: rubocop-rspec
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 description:
 email:
 - sarahmarie@hey.com
@@ -79,6 +107,7 @@ files:
 - lib/swagcov.rb
 - lib/swagcov/core_ext/string.rb
 - lib/swagcov/coverage.rb
+- lib/swagcov/dotfile.rb
 - lib/swagcov/railtie.rb
 - lib/swagcov/version.rb
 - lib/tasks/swagcove.rake