Acceptance tests for topology search and filter

hashicorp · Oct 17, 2022 · 56bd954 · 56bd954
1 parent a1f347e
commit 56bd954
Show file tree

Hide file tree

Showing 3 changed files with 34 additions and 2 deletions.
diff --git a/ui/app/templates/topology.hbs b/ui/app/templates/topology.hbs
@@ -466,8 +466,8 @@
             <div class="toolbar-item">
               {{#if this.model.nodes.length}}
                 <SearchBox
+                  @inputClass="node-search"
                   @searchTerm={{mut this.searchTerm}}
-                  {{!-- @onChange={{action this.buildTopology}} --}}
                   @placeholder="Search clients..."
                 />
               {{/if}}

diff --git a/ui/tests/acceptance/topology-test.js b/ui/tests/acceptance/topology-test.js
@@ -1,6 +1,6 @@
 /* eslint-disable qunit/require-expect */
 import { get } from '@ember/object';
-import { currentURL } from '@ember/test-helpers';
+import { currentURL, typeIn, click } from '@ember/test-helpers';
 import { module, test } from 'qunit';
 import { setupApplicationTest } from 'ember-qunit';
 import { setupMirage } from 'ember-cli-mirage/test-support';
@@ -311,4 +311,28 @@ module('Acceptance | topology', function (hooks) {
     assert.ok(Topology.filteredNodesWarning.isPresent);
     assert.ok(Topology.filteredNodesWarning.message.startsWith('1'));
   });
+
+  test('Filtering and Querying reduces the number of nodes shown', async function (assert) {
+    server.createList('node', 10);
+    server.createList('node', 2, {
+      nodeClass: 'foo-bar-baz',
+    });
+    server.createList('allocation', 5);
+
+    await Topology.visit();
+    assert.dom('[data-test-topo-viz-node]').exists({ count: 12 });
+
+    await typeIn('input.node-search', server.schema.nodes.first().name);
+    assert.dom('[data-test-topo-viz-node]').exists({ count: 1 });
+    await typeIn('input.node-search', server.schema.nodes.first().name);
+    assert.dom('[data-test-topo-viz-node]').doesNotExist();
+    await click('[title="Clear search"]');
+    assert.dom('[data-test-topo-viz-node]').exists({ count: 12 });
+
+    await Topology.facets.class.toggle();
+    await Topology.facets.class.options
+      .findOneBy('label', 'foo-bar-baz')
+      .toggle();
+    assert.dom('[data-test-topo-viz-node]').exists({ count: 2 });
+  });
 });
diff --git a/ui/tests/pages/topology.js b/ui/tests/pages/topology.js
@@ -8,6 +8,7 @@ import {
   visitable,
 } from 'ember-cli-page-object';
 
+import { multiFacet } from 'nomad-ui/tests/pages/components/facet';
 import TopoViz from 'nomad-ui/tests/pages/components/topo-viz';
 import notification from 'nomad-ui/tests/pages/components/notification';
 
@@ -19,6 +20,13 @@ export default create({
 
   viz: TopoViz('[data-test-topo-viz]'),
 
+  facets: {
+    datacenter: multiFacet('[data-test-datacenter-facet]'),
+    class: multiFacet('[data-test-class-facet]'),
+    state: multiFacet('[data-test-state-facet]'),
+    version: multiFacet('[data-test-version-facet]'),
+  },
+
   clusterInfoPanel: {
     scope: '[data-test-info-panel]',
     nodeCount: text('[data-test-node-count]'),