Fix test selection in Test::Integration::Mattermost

In ce3b81b9 we changed Test::Integration::Mattermost#perform to consume a required positional parameter and pass on the rest of the arguments to the superclass. There was an error in implementation here and instead of splatting the files array to produce no additional arguments, we passed in the default case an empty array. This had the effect of calling rspec with no test files, rather than expected default of 'qa/specs/features` Here we correct that bug, and re-add the :core suite, as that was a poor deuction in the investagtion for 74d6b8a2.

Fix test selection in Test::Integration::Mattermost
e1f03798 · Richard Clamp · 92249f1a · e1f03798
Commit e1f03798 authored Nov 08, 2017 by Richard Clamp
Hide whitespace changes
Inline Side-by-side

Showing with 2 additions and 2 deletions

mattermost.rb qa/qa/scenario/test/integration/mattermost.rb +2 -2

No files found.
--- a/qa/qa/scenario/test/integration/mattermost.rb
+++ b/qa/qa/scenario/test/integration/mattermost.rb
@@ -7,11 +7,11 @@ module QA
        # including staging and on-premises installation.
        #
        class Mattermost < Scenario::Entrypoint
-          tags :mattermost
+          tags :core, :mattermost

          def perform(address, mattermost, *files)
            Runtime::Scenario.mattermost = mattermost
-            super(address, files)
+            super(address, *files)
          end
        end
      end