Separate new tests from old tests

- the new tests use vue/test-utils and it was confusing to
  mix those tests in with existing ones, so I separated them
  out into their own describe block

spec/frontend/reports/components/report_section_spec.js

@@ -163,6 +163,20 @@ describe('Report section', () => {
           .catch(done.fail);
       });
 
+      it('is always expanded, if always-open is set to true', done => {
+        vm.alwaysOpen = true;
+        Vue.nextTick()
+          .then(() => {
+            expect(vm.$el.querySelector('.js-report-section-container')).not.toHaveCss(hiddenCss);
+            expect(vm.$el.querySelector('button')).toBeNull();
+          })
+          .then(done)
+          .catch(done.fail);
+      });
+    });
+  });
+
+  describe('snowplow events', () => {
     it('does emit an event on issue toggle if the shouldEmitToggleEvent prop does exist', done => {
       createComponent({ hasIssues: true, shouldEmitToggleEvent: true });
 
@@ -193,22 +207,18 @@ describe('Report section', () => {
         .catch(done.fail);
     });
 
-      it('is always expanded, if always-open is set to true', done => {
+    it('does not emit an event if always-open is set to true', done => {
       createComponent({ alwaysOpen: true, hasIssues: true, shouldEmitToggleEvent: true });
 
-        Vue.nextTick()
+      wrapper.vm
+        .$nextTick()
         .then(() => {
-            expect(wrapper.vm.$el.querySelector('.js-report-section-container')).not.toHaveCss(
-              hiddenCss,
-            );
-            expect(wrapper.vm.$el.querySelector('button')).toBeNull();
           expect(wrapper.emitted().toggleEvent).toBeUndefined();
         })
         .then(done)
         .catch(done.fail);
     });
   });
-  });
 
   describe('with failed request', () => {
     it('should render error indicator', () => {