Add auth user icon to dashboard list

jkocik · Oct 17, 2018 · da68899 · da68899
1 parent 447e73e
commit da68899
Show file tree

Hide file tree

Showing 3 changed files with 53 additions and 3 deletions.
diff --git a/src/components/buefy/BDashboardTable.vue b/src/components/buefy/BDashboardTable.vue
@@ -5,7 +5,7 @@
         extends: BTable,
         computed: {
             columnCount() {
-                return 11;
+                return 12;
             },
         },
     };

diff --git a/src/components/dashboard/DashboardTable.vue b/src/components/dashboard/DashboardTable.vue
@@ -72,6 +72,14 @@
                     <i class="fas fa-database has-text-primary"></i> {{ row.countQueries() }}
                 </div>
             </td>
+            <td class="tracker-summary auth">
+                <div v-show="row.isAuthProvided() && row.hasAuth()">
+                    <i class="fas fa-user has-text-success"></i>
+                </div>
+                <div v-show="row.isAuthProvided() && ! row.hasAuth()">
+                    <i class="fas fa-user-slash has-text-grey-lighter"></i>
+                </div>
+            </td>
 
             <td>
                 <div class="tags has-addons">

diff --git a/test/feature/specs/components/AppDashboard.spec.js b/test/feature/specs/components/AppDashboard.spec.js
@@ -29,7 +29,7 @@ describe('AppDashboard Component', () => {
         expect(wrapperTable.find('table tr:nth-child(1) td:nth-child(6)').text()).to.contain(tracker.path);
         expect(wrapperTable.find('table tr:nth-child(1) td:nth-child(7)').text()).to.contain(tracker.laravelExecutionTimeForHuman);
         expect(wrapperTable.find('table tr:nth-child(1) td:nth-child(8)').text()).to.contain(tracker.memoryUsageForHuman);
-        expect(wrapperTable.find('table tr:nth-child(1) td:nth-child(12)').text()).to.contain(tracker.laravelVersion);
+        expect(wrapperTable.find('table tr:nth-child(1) td:nth-child(13)').text()).to.contain(tracker.laravelVersion);
     });
 
     it('has memory usage mark as confusing when running tests', async () => {
@@ -133,6 +133,48 @@ describe('AppDashboard Component', () => {
         expect(wrapperTable.find('table tr:nth-child(1) .tracker-summary.queries').text()).that.is.empty;
     });
 
+    it('sees positive auth icon after data are delivered and user was logged in', async () => {
+        let wrapperTable = wrapper.find({ name: 'dashboard-table' });
+        let tracker = new Tracker(trackerFactory.create());
+        wrapper.vm.$store.commit('trackers/store', tracker);
+
+        wrapperTable.vm.$forceUpdate();
+        await wrapperTable.vm.$nextTick();
+        let positive = wrapperTable.find('table tr:nth-child(1) .tracker-summary.auth i.fa-user.has-text-success');
+        let negative = wrapperTable.find('table tr:nth-child(1) .tracker-summary.auth i.fa-user-slash.has-text-grey-lighter');
+        expect(positive.isVisible()).to.be.true;
+        expect(negative.isVisible()).to.be.false;
+    });
+
+    it('sees negative auth icon after data are delivered and user was not logged in', async () => {
+        let wrapperTable = wrapper.find({ name: 'dashboard-table' });
+        let trackerSource = trackerFactory.create('data', { auth: null });
+        let tracker = new Tracker(trackerSource);
+        wrapper.vm.$store.commit('trackers/store', tracker);
+
+        wrapperTable.vm.$forceUpdate();
+        await wrapperTable.vm.$nextTick();
+        let positive = wrapperTable.find('table tr:nth-child(1) .tracker-summary.auth i.fa-user.has-text-success');
+        let negative = wrapperTable.find('table tr:nth-child(1) .tracker-summary.auth i.fa-user-slash.has-text-grey-lighter');
+        expect(positive.isVisible()).to.be.false;
+        expect(negative.isVisible()).to.be.true;
+    });
+
+    it('does not see auth icon after data are delivered but auth is not provided at all', async () => {
+        let wrapperTable = wrapper.find({ name: 'dashboard-table' });
+        let trackerSource = trackerFactory.create('data', { auth: undefined });
+        delete trackerSource.data.auth;
+        let tracker = new Tracker(trackerSource);
+        wrapper.vm.$store.commit('trackers/store', tracker);
+
+        wrapperTable.vm.$forceUpdate();
+        await wrapperTable.vm.$nextTick();
+        let positive = wrapperTable.find('table tr:nth-child(1) .tracker-summary.auth i.fa-user.has-text-success');
+        let negative = wrapperTable.find('table tr:nth-child(1) .tracker-summary.auth i.fa-user-slash.has-text-grey-lighter');
+        expect(positive.isVisible()).to.be.false;
+        expect(negative.isVisible()).to.be.false;
+    });
+
     it('sees meta data of profiler in descending order', async () => {
         let trackerA = new Tracker(trackerFactory.create());
         let trackerB = new Tracker(trackerFactory.create('meta', { env: 'testing' }));
@@ -297,7 +339,7 @@ describe('AppDashboard Component', () => {
         let trDetailsA = wrapperTable.find('table tr:nth-child(1) + tr.detail');
         expect(trDetailsA.exists()).to.be.true;
         expect(trDetailsA.isVisible()).to.be.true;
-        expect(trDetailsA.find('td').attributes().colspan).to.equal('11');
+        expect(trDetailsA.find('td').attributes().colspan).to.equal('12');
 
         tr.trigger('click');
         let trDetailsABis = wrapperTable.find('table tr:nth-child(1) + tr.detail');