Commit 5069f0f2 authored by mfluharty's avatar mfluharty

Update spec to look for case name by ref

We can no longer use ('js-case-row').at(index).text() to check case name
since we're shallowMounting and case name is now inside a component
so we use a ref to find the case name instead
parent 2962ab8c
...@@ -81,7 +81,11 @@ export default { ...@@ -81,7 +81,11 @@ export default {
<div class="table-section section-20 section-wrap"> <div class="table-section section-20 section-wrap">
<div role="rowheader" class="table-mobile-header">{{ __('Name') }}</div> <div role="rowheader" class="table-mobile-header">{{ __('Name') }}</div>
<div class="table-mobile-content pr-md-1 gl-overflow-wrap-break"> <div class="table-mobile-content pr-md-1 gl-overflow-wrap-break">
<gl-friendly-wrap :symbols="$options.wrapSymbols" :text="testCase.name" /> <gl-friendly-wrap
ref="casename"
:symbols="$options.wrapSymbols"
:text="testCase.name"
/>
</div> </div>
</div> </div>
......
...@@ -23,6 +23,7 @@ describe('Test reports suite table', () => { ...@@ -23,6 +23,7 @@ describe('Test reports suite table', () => {
const noCasesMessage = () => wrapper.find('.js-no-test-cases'); const noCasesMessage = () => wrapper.find('.js-no-test-cases');
const allCaseRows = () => wrapper.findAll('.js-case-row'); const allCaseRows = () => wrapper.findAll('.js-case-row');
const findCaseRowAtIndex = index => wrapper.findAll('.js-case-row').at(index); const findCaseRowAtIndex = index => wrapper.findAll('.js-case-row').at(index);
const findCaseNameAtIndex = index => wrapper.findAll({ ref: 'casename' }).at(index);
const findIconForRow = (row, status) => row.find(`.ci-status-icon-${status}`); const findIconForRow = (row, status) => row.find(`.ci-status-icon-${status}`);
const createComponent = (suite = testSuite) => { const createComponent = (suite = testSuite) => {
...@@ -70,9 +71,9 @@ describe('Test reports suite table', () => { ...@@ -70,9 +71,9 @@ describe('Test reports suite table', () => {
.filter(x => x.status === TestStatus.SKIPPED) .filter(x => x.status === TestStatus.SKIPPED)
.map(x => x.name); .map(x => x.name);
expect(findCaseRowAtIndex(0).text()).toContain(failedCaseNames[0]); expect(findCaseNameAtIndex(0).attributes('text')).toEqual(failedCaseNames[0]);
expect(findCaseRowAtIndex(1).text()).toContain(failedCaseNames[1]); expect(findCaseNameAtIndex(1).attributes('text')).toEqual(failedCaseNames[1]);
expect(findCaseRowAtIndex(2).text()).toContain(skippedCaseNames[0]); expect(findCaseNameAtIndex(2).attributes('text')).toEqual(skippedCaseNames[0]);
}); });
it('renders the correct icon for each status', () => { it('renders the correct icon for each status', () => {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment