gergich 1.1.1 → 2.0.0

data/spec/gergich/capture/eslint_capture_spec.rb

@@ -1,31 +0,0 @@
-# frozen_string_literal: true
-
-require_relative "../../support/capture_shared_examples"
-
-RSpec.describe Gergich::Capture::EslintCapture do
-  let(:output) do
-    <<~OUTPUT
-      jsapp/models/user.js
-        4:21  error    Missing semicolon  semi
-        5:1   warning  Too much cowbell   cowbell-overload
-    OUTPUT
-  end
-  let(:comments) do
-    [
-      {
-        path: "jsapp/models/user.js",
-        position: 4,
-        message: "[eslint] Missing semicolon",
-        severity: "error"
-      },
-      {
-        path: "jsapp/models/user.js",
-        position: 5,
-        message: "[eslint] Too much cowbell",
-        severity: "warn"
-      }
-    ]
-  end
-
-  it_behaves_like "a captor"
-end

data/spec/gergich/capture/flake8_capture_spec.rb

@@ -1,23 +0,0 @@
-# frozen_string_literal: true
-
-require_relative "../../support/capture_shared_examples"
-
-RSpec.describe Gergich::Capture::Flake8Capture do
-  let(:output) do
-    <<~OUTPUT
-      ./djangogeneric/settings/base.py:73:80: E501 line too long (81 > 79 characters)
-    OUTPUT
-  end
-  let(:comments) do
-    [
-      {
-        path: "./djangogeneric/settings/base.py",
-        position: 73,
-        message: "[flake8] E501 line too long (81 > 79 characters)",
-        severity: "error"
-      }
-    ]
-  end
-
-  it_behaves_like "a captor"
-end

data/spec/gergich/capture/i18nliner_capture_spec.rb

@@ -1,25 +0,0 @@
-# frozen_string_literal: true
-
-require_relative "../../support/capture_shared_examples"
-
-RSpec.describe Gergich::Capture::I18nlinerCapture do
-  let(:output) do
-    <<~OUTPUT
-      1)
-      invalid signature on line 4: <unsupported expression>
-      jsapp/models/user.js
-    OUTPUT
-  end
-  let(:comments) do
-    [
-      {
-        path: "jsapp/models/user.js",
-        position: 4,
-        message: "[i18n] invalid signature: <unsupported expression>",
-        severity: "error"
-      }
-    ]
-  end
-
-  it_behaves_like "a captor"
-end

data/spec/gergich/capture/rubocop_capture_spec.rb

@@ -1,66 +0,0 @@
-# frozen_string_literal: true
-
-require_relative "../../support/capture_shared_examples"
-
-RSpec.describe Gergich::Capture::RubocopCapture do
-  let(:output) do
-    <<~OUTPUT
-      Offenses:
-
-      bin/gergich:47:8: C: Prefer double-quoted strings
-      if ENV['DEBUG']
-             ^^^^^^^
-      foo/bar/baz.rb:1:2: W: no context for this one :shrug:
-      lib/gergich.rb:10:9: E: this is a terrible name
-
-      seriously, what were you thinking?
-          def foo
-              ^^^
-      lib/gergich.rb:22:55: W: Line is too long. [55/54]
-          def initialize(ref = "HEAD", revision_number = nil)
-                                                           ^^
-    OUTPUT
-  end
-  let(:comments) do
-    [
-      {
-        path: "bin/gergich",
-        position: 47,
-        message: "[rubocop] Prefer double-quoted strings\n\n if ENV['DEBUG']\n        ^^^^^^^\n",
-        severity: "info"
-      },
-      {
-        path: "foo/bar/baz.rb",
-        position: 1,
-        message: "[rubocop] no context for this one :shrug:\n",
-        severity: "warn"
-      },
-      {
-        path: "lib/gergich.rb",
-        position: 10,
-        message: <<~OUTPUT,
-          [rubocop] this is a terrible name
-
-          seriously, what were you thinking?
-
-               def foo
-                   ^^^
-        OUTPUT
-        severity: "error"
-      },
-      {
-        path: "lib/gergich.rb",
-        position: 22,
-        message: <<~OUTPUT,
-          [rubocop] Line is too long. [55/54]
-
-               def initialize(ref = "HEAD", revision_number = nil)
-                                                                ^^
-        OUTPUT
-        severity: "warn"
-      }
-    ]
-  end
-
-  it_behaves_like "a captor"
-end

data/spec/gergich/capture/shellcheck_capture_spec.rb

@@ -1,83 +0,0 @@
-# frozen_string_literal: true
-
-require_relative "../../support/capture_shared_examples"
-
-RSpec.describe Gergich::Capture::ShellcheckCapture do
-  let(:output) do
-    <<~'OUTPUT'
-      [
-        {
-          "file": "bin/sync-translations.sh",
-          "line": 23,
-          "endLine": 23,
-          "column": 21,
-          "endColumn": 21,
-          "level": "style",
-          "code": 2006,
-          "message": "Use $(..) instead of legacy `..`."
-        },
-        {
-          "file": "bin/sync-translations.sh",
-          "line": 23,
-          "endLine": 23,
-          "column": 43,
-          "endColumn": 43,
-          "level": "warning",
-          "code": 2046,
-          "message": "Quote this to prevent word splitting."
-        },
-        {
-          "file": "bin/sync-translations.sh",
-          "line": 32,
-          "endLine": 32,
-          "column": 62,
-          "endColumn": 62,
-          "level": "info",
-          "code": 2086,
-          "message": "Double quote to prevent globbing and word splitting."
-        },
-        {
-          "file": "fail.sh",
-          "line": 3,
-          "endLine": 3,
-          "column": 12,
-          "endColumn": 12,
-          "level": "error",
-          "code": 1101,
-          "message": "Delete trailing spaces after \\ to break line (or use quotes for literal space)."
-        }
-      ]
-    OUTPUT
-  end
-
-  let(:comments) do
-    [
-      {
-        path: "bin/sync-translations.sh",
-        position: 23,
-        message: "Use $(..) instead of legacy `..`.",
-        severity: "info"
-      },
-      {
-        path: "bin/sync-translations.sh",
-        position: 23,
-        message: "Quote this to prevent word splitting.",
-        severity: "warn"
-      },
-      {
-        path: "bin/sync-translations.sh",
-        position: 32,
-        message: "Double quote to prevent globbing and word splitting.",
-        severity: "info"
-      },
-      {
-        path: "fail.sh",
-        position: 3,
-        message: "Delete trailing spaces after \\ to break line (or use quotes for literal space).",
-        severity: "error"
-      }
-    ]
-  end
-
-  it_behaves_like "a captor"
-end

data/spec/gergich/capture/stylelint_capture_spec.rb

@@ -1,54 +0,0 @@
-# frozen_string_literal: true
-
-require_relative "../../support/capture_shared_examples"
-
-RSpec.describe Gergich::Capture::StylelintCapture do
-  let(:output) do
-    <<~OUTPUT
-      app/stylesheets/base/_print.scss
-       3:17  ✖  Unexpected invalid hex color "#owiehfi"   color-no-invalid-hex
-       3:17  ⚠  Expected "#owiehfi" to be "#OWIEHFI"      color-hex-case
-
-      app/stylesheets/base/_variables.scss
-        2:15  ✖  Unexpected invalid hex color "#2D3B4"   color-no-invalid-hex
-       30:15  ⚠  Expected "#2d3b4a" to be "#2D3B4A"      color-hex-case
-       45:12  ℹ  Expected "#2d3b4a" to be "#2D3B4A"      color-hex-case
-    OUTPUT
-  end
-  let(:comments) do
-    [
-      {
-        path: "app/stylesheets/base/_print.scss",
-        position: 3,
-        message: "[stylelint] Unexpected invalid hex color \"#owiehfi\"",
-        severity: "error"
-      },
-      {
-        path: "app/stylesheets/base/_print.scss",
-        position: 3,
-        message: "[stylelint] Expected \"#owiehfi\" to be \"#OWIEHFI\"",
-        severity: "warn"
-      },
-      {
-        path: "app/stylesheets/base/_variables.scss",
-        position: 2,
-        message: "[stylelint] Unexpected invalid hex color \"#2D3B4\"",
-        severity: "error"
-      },
-      {
-        path: "app/stylesheets/base/_variables.scss",
-        position: 30,
-        message: "[stylelint] Expected \"#2d3b4a\" to be \"#2D3B4A\"",
-        severity: "warn"
-      },
-      {
-        path: "app/stylesheets/base/_variables.scss",
-        position: 45,
-        message: "[stylelint] Expected \"#2d3b4a\" to be \"#2D3B4A\"",
-        severity: "info"
-      }
-    ]
-  end
-
-  it_behaves_like "a captor"
-end

data/spec/gergich/capture/swiftlint_capture_spec.rb

@@ -1,42 +0,0 @@
-# frozen_string_literal: true
-
-require_relative "../../support/capture_shared_examples"
-
-RSpec.describe Gergich::Capture::SwiftlintCapture do
-  # rubocop:disable Metrics/LineLength
-  let(:colon_violation) { "Colon Violation: Colons should be next to the identifier when specifying a type. (colon)" }
-  let(:line_length_violation) { "Line Length Violation: Line should be 100 characters or less: currently 129 characters (line_length)" }
-  let(:force_cast_violation) { "Force Cast Violation: Force casts should be avoided. (force_cast)" }
-  # rubocop:enable Metrics/LineLength
-  let(:output) do
-    <<~OUTPUT
-      /path/to/My.swift:13:22: warning: #{colon_violation}
-      /path/to/Fail.swift:76: warning: #{line_length_violation}
-      /path/to/Cast.swift:15:9: error: #{force_cast_violation}
-    OUTPUT
-  end
-  let(:comments) do
-    [
-      {
-        path: "/path/to/My.swift",
-        position: 13,
-        message: "[swiftlint] #{colon_violation}",
-        severity: "warn"
-      },
-      {
-        path: "/path/to/Fail.swift",
-        position: 76,
-        message: "[swiftlint] #{line_length_violation}",
-        severity: "warn"
-      },
-      {
-        path: "/path/to/Cast.swift",
-        position: 15,
-        message: "[swiftlint] #{force_cast_violation}",
-        severity: "error"
-      }
-    ]
-  end
-
-  it_behaves_like "a captor"
-end

data/spec/gergich/capture_spec.rb

@@ -1,75 +0,0 @@
-# frozen_string_literal: true
-
-require_relative "../../lib/gergich/capture"
-
-RSpec.describe Gergich::Capture do
-  let!(:draft) { double }
-
-  let :output do
-    <<~OUTPUT
-      #{path}
-        4:21  error  Missing semicolon  semi
-    OUTPUT
-  end
-
-  before do
-    allow(Gergich::Draft).to receive(:new).and_return(draft)
-    $stdin = StringIO.new(output)
-  end
-
-  after do
-    $stdin = STDIN
-  end
-
-  context "stdin" do
-    let(:path) { "jsapp/models/user.js" }
-
-    it "should catch errors" do
-      expect(draft).to receive(:add_comment).with(
-        "jsapp/models/user.js",
-        4,
-        "[eslint] Missing semicolon",
-        "error"
-      )
-      described_class.run("eslint", "-", suppress_output: true)
-    end
-
-    it "shouldn't eat stdin" do
-      allow(draft).to receive(:add_comment)
-      expect($stdout).to receive(:puts).exactly(output.lines.size).times
-      described_class.run("eslint", "-")
-    end
-  end
-
-  context "absolute paths" do
-    before do
-      allow(described_class).to receive(:base_path).and_return("/the/directory/")
-    end
-
-    context "under us" do
-      let(:path) { "/the/directory/jsapp/models/user.js" }
-      it "should be relativized" do
-        expect(draft).to receive(:add_comment).with(
-          "jsapp/models/user.js",
-          4,
-          "[eslint] Missing semicolon",
-          "error"
-        )
-        described_class.run("eslint", "-", suppress_output: true)
-      end
-    end
-
-    context "elsewhere" do
-      let(:path) { "/other/directory/jsapp/models/user.js" }
-      it "should not be relativized" do
-        expect(draft).to receive(:add_comment).with(
-          "/other/directory/jsapp/models/user.js",
-          4,
-          "[eslint] Missing semicolon",
-          "error"
-        )
-        described_class.run("eslint", "-", suppress_output: true)
-      end
-    end
-  end
-end