Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Sidebranch: component and acceptance tests #8903

Merged
merged 38 commits into from
May 7, 2020
Merged
Show file tree
Hide file tree
Changes from 28 commits
Commits
Show all changes
38 commits
Select commit Hold shift + click to select a range
40e2636
address secondary card overflow issue
Monkeychip Apr 29, 2020
ac8a859
setup replicaiton header test
Monkeychip Apr 29, 2020
5099118
address secondary card overflow issue
Monkeychip Apr 29, 2020
cc07e0d
setup replication secondary card test
Monkeychip Apr 29, 2020
eccfbf7
setup replicaiton header test
Monkeychip Apr 29, 2020
5f4204b
setup replicaiton page test
Monkeychip Apr 29, 2020
1af6828
setup replication secondary card test
Monkeychip Apr 29, 2020
e8961e5
setup replication dashboard test
Monkeychip Apr 29, 2020
d4988f8
setup replicaiton page test
Monkeychip Apr 29, 2020
4d73d53
remove unused code
Monkeychip Apr 29, 2020
b4a6f10
fix overflow
Monkeychip Apr 29, 2020
fe598c9
finish test for rep dashboard
Monkeychip Apr 30, 2020
fe75932
update rep secondary card test
Monkeychip Apr 30, 2020
7711ef2
finish rep header test
Monkeychip Apr 30, 2020
46bc438
fix rep table rows and header test
Monkeychip Apr 30, 2020
06a01e0
fix header test
Monkeychip Apr 30, 2020
83b0d00
fix missing data-test-primary-cluster
Monkeychip Apr 30, 2020
2dbf648
add to secondary test
Monkeychip May 1, 2020
67a9e12
remove pauseTest
Monkeychip May 1, 2020
b8c6eab
add to enterprise replication test
Monkeychip May 1, 2020
0dd8565
add mode to dr secondary test
Monkeychip May 1, 2020
dc82f7c
remove pauseTest
Monkeychip May 1, 2020
2a13177
add enterprise to test
Monkeychip May 4, 2020
2e63a22
amend per pr commments
Monkeychip May 6, 2020
cd4d33e
re organize rep secondary card test
Monkeychip May 6, 2020
8ef20d2
adjust error heights with design input
Monkeychip May 6, 2020
6d8b6ea
move const around in rep secondary card test
Monkeychip May 6, 2020
4c2cc33
move const around and message for rep dashboard test
Monkeychip May 6, 2020
fb4e4d3
amend per pr review comments
Monkeychip May 6, 2020
836b582
remove styling from grid-item-left
Monkeychip May 6, 2020
a828256
Merge branch 'ui/replication-status-discoverability' into ui/tests-v2
Monkeychip May 6, 2020
83e0cc9
remove dup hasErrorClass key
Monkeychip May 6, 2020
c62814e
Merge branch 'ui/tests-v2' of github.com:hashicorp/vault into ui/test…
Monkeychip May 6, 2020
cff715f
quick fix
Monkeychip May 6, 2020
3d7d01b
test failure fix
Monkeychip May 6, 2020
50955f5
fix test due to merge
Monkeychip May 6, 2020
6b37e6c
remove hasErrorClass
Monkeychip May 6, 2020
6cc801b
modify test message
Monkeychip May 7, 2020
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
10 changes: 6 additions & 4 deletions ui/app/styles/components/replication-dashboard.scss
Original file line number Diff line number Diff line change
Expand Up @@ -23,14 +23,12 @@

.card-container {
display: grid;
grid-gap: 2rem;
grid-gap: 1rem;
grid-template-columns: 1fr 1fr;
grid-template-rows: 1fr 1fr 0.8fr;
grid-template-rows: 0.2fr 1fr 0.2fr;
padding: $spacing-l;
line-height: 1.5;

height: 230px;

@include until(1320px) {
// prevent an issue with the card descriptions wrapping and expanding height
min-height: 250px;
Expand All @@ -43,6 +41,10 @@
.grid-item-left {
grid-column: 1/1;
grid-row: 2/2;

height: 75px;
max-height: 75px;
overflow: auto;
Monkeychip marked this conversation as resolved.
Show resolved Hide resolved
}
.grid-item-right {
grid-column: 2/2;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@
/>
{{#if Page.isDisabled}}
<EmptyState
@title="Disaster Recovery secondary not set up"
@title="{{Page.replicationMode}} secondary not set up"
@message={{Page.message}}
@icon="alert-circle-outline"
@bottomBorder={{true}}
Expand Down
1 change: 0 additions & 1 deletion ui/lib/core/addon/components/replication-dashboard.js
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,6 @@ export default Component.extend({
data: null,
replicationDetails: null,
isSecondary: null,
dr: null,
isSyncing: computed('replicationDetails', 'isSecondary', function() {
const { state } = this.replicationDetails;
const isSecondary = this.isSecondary;
Expand Down
12 changes: 1 addition & 11 deletions ui/lib/core/addon/components/replication-header.js
Original file line number Diff line number Diff line change
@@ -1,18 +1,8 @@
/**
* @module ReplicationHeader
* ARG TODO: finish
*
* @example
* ```js
* <ReplicationHeader finish/>
* ```
*/

import Component from '@ember/component';
import layout from '../templates/components/replication-header';

export default Component.extend({
layout,
data: null,
isSecondary: null,
'data-test-replication-header': true,
});
16 changes: 15 additions & 1 deletion ui/lib/core/addon/components/replication-secondary-card.js
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,21 @@ import { computed } from '@ember/object';
import layout from '../templates/components/replication-secondary-card';
import { clusterStates } from 'core/helpers/cluster-states';

/**
* @module ReplicationSecondaryCard
* ReplicationSecondaryCard components
*
* @example
* ```js
* <ReplicationSecondaryCard
@title='States'
@replicationDetails=replicationDetails
/>
* ```
* @param {string} [title=null] - The title to be displayed on the top left corner of the card.
* @param replicationDetails=null{DS.Model.replicationDetails} - An Ember data object off the Ember data model. It is computed at the parent component and passed through to this component.
*/

export default Component.extend({
layout,
title: null,
Expand Down Expand Up @@ -31,7 +46,6 @@ export default Component.extend({
// we should use the clusterStates helper instead
return this.state === 'stream-wals';
}),

hasErrorClass: computed('replicationDetails', 'title', 'state', 'connection', function() {
const { title, state, connection } = this;

Expand Down
2 changes: 1 addition & 1 deletion ui/lib/core/addon/templates/components/alert-inline.hbs
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,6 @@
@glyph={{this.alertType.glyph}}
class={{this.alertType.glyphClass}}
/>
<p class="{{this.textClass}}">
<p class="{{this.textClass}}" data-test-inline-error-message>
{{@message}}
</p>
11 changes: 6 additions & 5 deletions ui/lib/core/addon/templates/components/replication-dashboard.hbs
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
<div class="replication-dashboard box is-sideless is-fullwidth is-marginless">
<div class="replication-dashboard box is-sideless is-fullwidth is-marginless" data-test-replication-dashboard>
{{#if isSecondary}}
<h2 class="has-text-weight-semibold is-size-4 has-bottom-margin-xs">
Primary Cluster
Expand All @@ -7,7 +7,7 @@
If you set a primary_cluster_addr when enabling replication, it will appear here.
</p>
{{#if replicationDetails.primaryClusterAddr}}
<code>{{replicationDetails.primaryClusterAddr}}</code>
<code data-test-primary-cluster-address>{{replicationDetails.primaryClusterAddr}}</code>
{{else}}
<code>no known primary cluster address</code>
{{/if}}
Expand Down Expand Up @@ -37,10 +37,11 @@
@type="info"
@secondIconType="loading"
@message="The cluster is syncing. This happens when the secondary is too far behind the primary to use the normal stream-wals state for catching up."
@class="has-top-margin-xl" />
@class="has-top-margin-xl"
data-test-isSyncing/>
{{/if}}
<ReplicationTableRows
@replicationDetails={{replicationDetails}}
@clusterMode={{clusterMode}} />
<ReplicationDocLink />
@clusterMode={{clusterMode}}/>
<ReplicationDocLink data-test-replication-doc-link/>
</div>
6 changes: 3 additions & 3 deletions ui/lib/core/addon/templates/components/replication-header.hbs
Original file line number Diff line number Diff line change
Expand Up @@ -33,18 +33,18 @@
<p.levelLeft>
<h1 class="title is-3">
{{title}}
<span class="tag">
<span class="tag" data-test-mode>
{{if isSecondary 'Secondary' 'Primary'}}
</span>
<span class="tag">
<span class="tag" data-test-clusterId>
{{data.dr.clusterIdDisplay}}
</span>
</h1>
</p.levelLeft>
</PageHeader>

{{#if showTabs}}
<div class="tabs-container box is-bottomless is-marginless is-fullwidth is-paddingless">
<div class="tabs-container box is-bottomless is-marginless is-fullwidth is-paddingless" data-test-tabs>
<nav class="tabs">
<ul>
{{#link-to
Expand Down
3 changes: 2 additions & 1 deletion ui/lib/core/addon/templates/components/replication-page.hbs
Original file line number Diff line number Diff line change
@@ -1,8 +1,9 @@
<div class="replication-page">
<div class="replication-page" data-test-replication-page>
{{yield (hash
header=(component 'replication-header' data=model title=replicationMode isSecondary=isSecondary)
toggle=(component 'replication-toggle')
dashboard=(component 'replication-dashboard' data=model isSecondary=isSecondary replicationDetails=replicationDetails clusterMode=clusterMode)
replicationMode=replicationMode
isDisabled=isDisabled
message=message
)}}
Expand Down
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
<div class="selectable-card is-rounded card-container {{if hasErrorClass "has-border-danger"}}">
<div class="selectable-card is-rounded card-container {{if hasErrorClass "has-border-danger"}}" data-test-replication-secondary-card>
{{!-- Check if State or WAL Card --}}
{{#if (eq title "States")}}
<h3 class="title is-5 grid-item-title card-title">{{title}}</h3>
Expand All @@ -7,20 +7,20 @@
<h6 class="title is-6">state</h6>
<p class="has-text-grey is-size-8">How this cluster is communicating with others at this moment.</p>
{{else}}
<h6 class="has-text-danger">state</h6>
<h6 class="has-text-danger" data-test-error>state</h6>
<AlertInline
@type="danger"
@message="Please check your server logs." />
{{/if}}
</div>
<div class="grid-item-right">
{{#if (eq connection "transient_failure")}}
<h6 class="title is-6 has-text-danger">connection_state</h6>
<h6 class="title is-6 has-text-danger" data-test-error>connection_state</h6>
<AlertInline
@type="danger"
@message="There has been some transient failure. Your cluster will eventually switch back to connection and try to establish a connection again." />
{{else if (eq connection "shutdown")}}
<h6 class="title is-6 has-text-danger">connection_state</h6>
<h6 class="title is-6 has-text-danger" data-test-error>connection_state</h6>
<AlertInline
@type="danger"
@message="Your connection has shut down. This may be because the application explicitly requested a shutdown or a non-recoverable error has happened during attempts to communicate." />
Expand All @@ -29,11 +29,11 @@
<p class="has-text-grey is-size-8">The health of the connection between this cluster and others.</p>
{{/if}}
</div>
<h2 class="title is-3 grid-item-left-bottom">{{state}}
<h2 class="title is-3 grid-item-left-bottom" data-test-state>{{state}}
{{#if inSyncState }}
<ToolTip @verticalPosition="above" as |T|>
<T.trigger>
<Icon @glyph={{"check-circle-outline" }} @size="m" class={{"has-text-success"}} />
<Icon @glyph={{"check-circle-outline" }} @size="m" class={{"has-text-success"}} data-test-glyph />
</T.trigger>
<T.content @class="tool-tip">
<div class="box">
Expand All @@ -43,7 +43,7 @@
</ToolTip>
{{/if}}
</h2>
<h2 class="title is-3 grid-item-right-bottom">{{connection}}</h2>
<h2 class="title is-3 grid-item-right-bottom" data-test-connection>{{connection}}</h2>
{{else}}
<h3 class="title is-5 grid-item-title card-title">{{title}}
<ToolTip @verticalPosition="above" @closeDelay="2000ms" as |T|>
Expand All @@ -67,12 +67,12 @@
</div>
<div class="grid-item-right">
<h6 class="title is-6">last_wal</h6>
<p class="has-text-grey is-size-8">{{lastWAL}}</p>
<p class="has-text-grey is-size-8" data-test-lastWAL>{{lastWAL}}</p>
</div>
<code class="title is-3 grid-item-left-bottom">{{format-number delta}}</code>
<code class="title is-3 grid-item-left-bottom" data-test-delta>{{format-number delta}}</code>
<div class="grid-item-right-bottom">
<h6 class="title is-6">last_remote_wal</h6>
<p class="has-text-grey is-size-8">{{lastRemoteWAL}}</p>
<p class="has-text-grey is-size-8" data-test-lastRemoteWAL>{{lastRemoteWAL}}</p>
</div>
{{/if}}
</div>
49 changes: 49 additions & 0 deletions ui/tests/acceptance/enterprise-replication-test.js
Original file line number Diff line number Diff line change
Expand Up @@ -49,6 +49,19 @@ module('Acceptance | Enterprise | replication', function(hooks) {
const mode = 'deny';
let mountPath;

// confirm unable to visit dr secondary details page when both replications are disabled
await visit('/vault/replication-dr-promote/details');
assert.dom('[data-test-component="empty-state"]').exists();
Monkeychip marked this conversation as resolved.
Show resolved Hide resolved
assert
.dom('[data-test-empty-state-title]')
.includesText('Disaster Recovery secondary not set up', 'shows the correct title of the empty state');

assert.equal(
find('[data-test-empty-state-message]').textContent.trim(),
'This cluster has not been enabled as a Disaster Recovery Secondary. You can do so by enabling replication and adding a secondary from the Disaster Recovery Primary.',
'renders default message specific to when no replication is enabled'
);

await visit('/vault/replication');
assert.equal(currentURL(), '/vault/replication');

Expand All @@ -58,6 +71,10 @@ module('Acceptance | Enterprise | replication', function(hooks) {

await click('[data-test-replication-enable]');
await pollCluster(this.owner);
await settled();

// confirm that the details dashboard shows
assert.dom('[data-test-replication-dashboard]').exists();

// add a secondary with a mount filter config
await click('[data-test-replication-link="secondaries"]');
Expand Down Expand Up @@ -104,6 +121,13 @@ module('Acceptance | Enterprise | replication', function(hooks) {
`/vault/replication/performance/secondaries`,
'redirects to the secondaries page'
);
// nav back to details page and confirm secondary is in the know secondaries table
await click('[data-test-replication-link="details"]');
await settled();
assert
.dom(`[data-test-secondaries-row=${secondaryName}]`)
Monkeychip marked this conversation as resolved.
Show resolved Hide resolved
.exists('shows a table row the recently added secondary');
// ARG TODO when get connected state check here

// nav to DR
await visit('/vault/replication/dr');
Expand All @@ -123,6 +147,15 @@ module('Acceptance | Enterprise | replication', function(hooks) {
await click('button[type="submit"]');

await pollCluster(this.owner);
// empty state inside of know secondaries table
assert.dom('[data-test-empty-state-title]').exists();
Monkeychip marked this conversation as resolved.
Show resolved Hide resolved
assert
.dom('[data-test-empty-state-title]')
.includesText(
'No known dr secondary clusters associated with this cluster',
'shows the correct title of the empty state'
);

assert.ok(
find('[data-test-replication-title]').textContent.includes('Disaster Recovery'),
'it displays the replication type correctly'
Expand Down Expand Up @@ -161,4 +194,20 @@ module('Acceptance | Enterprise | replication', function(hooks) {
await visit('/vault/replication/dr/manage');
assert.ok(findAll('[data-test-demote-warning]').length, 'displays the demotion warning');
});

test('navigating to dr secondary details page', async function(assert) {
// enable dr replication
await visit('/vault/replication/dr');
await fillIn('[data-test-replication-cluster-mode-select]', 'primary');
await click('[data-test-replication-enable]');
await pollCluster(this.owner);

await visit('/vault/replication-dr-promote/details');
assert.dom('[data-test-component="empty-state"]').exists();
Monkeychip marked this conversation as resolved.
Show resolved Hide resolved
assert.equal(
find('[data-test-empty-state-message]').textContent.trim(),
'This Disaster Recovery secondary has not been enabled. You can do so from the Disaster Recovery Primary.',
'renders default message specific to when no replication is enabled'
);
});
});
2 changes: 1 addition & 1 deletion ui/tests/acceptance/replication/dr/secondary-test.js
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ import { currentURL, visit } from '@ember/test-helpers';

import authPage from 'vault/tests/pages/auth';

module('Acceptance | DR secondary details', function(hooks) {
module('Acceptance| Enterprise | DR secondary details', function(hooks) {
setupApplicationTest(hooks);

hooks.beforeEach(function() {
Expand Down
58 changes: 58 additions & 0 deletions ui/tests/integration/components/replication-dashboard-test.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,58 @@
import { module, test } from 'qunit';
import { setupRenderingTest } from 'ember-qunit';
import { render } from '@ember/test-helpers';
import hbs from 'htmlbars-inline-precompile';

const REPLICATION_DETAILS = {
state: 'stream-wals',
primaryClusterAddr: 'https://127.0.0.1:8201',
};

const REPLICATION_DETAILS_SYNCING = {
state: 'merkle-diff',
primaryClusterAddr: 'https://127.0.0.1:8201',
};

module('Integration | Enterprise | Component | replication-dashboard', function(hooks) {
setupRenderingTest(hooks);

hooks.beforeEach(function() {
this.set('clusterMode', 'secondary');
this.set('isSecondary', true);
});

test('it renders', async function(assert) {
this.set('replicationDetails', REPLICATION_DETAILS);

await render(hbs`<ReplicationDashboard
@replicationDetails={{replicationDetails}}
@clusterMode={{clusterMode}}
@isSecondary={{isSecondary}}
/>`);

assert.dom('[data-test-replication-dashboard]').exists();
assert.dom('[data-test-table-rows').exists();
assert
.dom('[data-test-primary-cluster-address]')
Monkeychip marked this conversation as resolved.
Show resolved Hide resolved
.includesText(
REPLICATION_DETAILS.primaryClusterAddr,
`shows the correct primary cluster address value`
);
assert.dom('[data-test-replication-doc-link]').exists();
assert.dom('[data-test-flash-message]').doesNotExist('no flash message is displayed on render');
});

test('it renders an alert banner if the dashboard is syncing', async function(assert) {
this.set('componentToRender', 'replication-secondary-card');
this.set('replicationDetailsSyncing', REPLICATION_DETAILS_SYNCING);

await render(hbs`<ReplicationDashboard
@replicationDetails={{replicationDetailsSyncing}}
@clusterMode={{clusterMode}}
@isSecondary={{isSecondary}}
@componentToRender={{componentToRender}}
Monkeychip marked this conversation as resolved.
Show resolved Hide resolved
/>`);

assert.dom('[data-test-isSyncing]').exists();
});
});
Loading