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

[#2098] Add show more button for error messages #2105

Merged
merged 13 commits into from
Mar 14, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
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
4 changes: 4 additions & 0 deletions frontend/cypress/config/repo-config.csv
Original file line number Diff line number Diff line change
@@ -1,3 +1,7 @@
Repository's Location,Branch,File formats,Ignore Glob List,Ignore standalone config,Ignore Commits List,Ignore Authors List,Shallow Cloning,Find Previous Authors
https://github.com/reposense/RepoSense.git,cypress,,,,,,,
https://github.com/reposense/testrepo-Empty.git,master,,,,,,,
https://github.com/reposense/testrepo-Empty2.git,master,,,,,,,
https://github.com/reposense/testrepo-Empty3.git,master,,,,,,,
https://github.com/reposense/testrepo-Empty4.git,master,,,,,,,
https://github.com/reposense/testrepo-Empty5.git,master,,,,,,,
Original file line number Diff line number Diff line change
Expand Up @@ -27,4 +27,24 @@ describe('error summary', () => {
cy.get('.error-message-box')
.should('not.be.visible');
});

it('can be expanded and collapsed if count > 4', () => {
cy.get('.error-message-box')
.find('.error-message-box__failed-repo')
.should('have.length', 4);

cy.get('.error-message-box__show-more-container > a')
.click();

cy.get('.error-message-box')
.find('.error-message-box__failed-repo')
.should('have.length', 5);

cy.get('.error-message-box__show-more-container > a')
.click();

cy.get('.error-message-box')
.find('.error-message-box__failed-repo')
.should('have.length', 4);
});
});
14 changes: 7 additions & 7 deletions frontend/cypress/tests/chartView/chartView_zoomFeature.cy.js
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ describe('zoom features in code view', () => {
const zoomKey = Cypress.platform === 'darwin' ? '{meta}' : '{ctrl}';
it('click on view commits button', () => {
cy.get('.icon-button.fa-list-ul')
.should('be.visible')
.should('exist')
.first()
.click();

Expand Down Expand Up @@ -57,7 +57,7 @@ describe('date changes in chart view should reflect in zoom', () => {
.type('2018-06-11');

cy.get('.icon-button.fa-list-ul')
.should('be.visible')
.should('exist')
.first()
.click();

Expand All @@ -84,7 +84,7 @@ describe('date changes in chart view should reflect in zoom', () => {
.type('2018-05-20');

cy.get('.icon-button.fa-list-ul')
.should('be.visible')
.should('exist')
.first()
.click();

Expand Down Expand Up @@ -112,7 +112,7 @@ describe('date changes in chart view should reflect in zoom', () => {
.type('2019-08-19');

cy.get('.icon-button.fa-list-ul')
.should('be.visible')
.should('exist')
.first()
.click();

Expand Down Expand Up @@ -140,7 +140,7 @@ describe('date changes in chart view should reflect in zoom', () => {
.type('2018-07-20');

cy.get('.icon-button.fa-list-ul')
.should('be.visible')
.should('exist')
.first()
.click();

Expand Down Expand Up @@ -171,7 +171,7 @@ describe('date changes in chart view should reflect in zoom', () => {
.type('2019-03-09');

cy.get('.icon-button.fa-list-ul')
.should('be.visible')
.should('exist')
.first()
.click();

Expand Down Expand Up @@ -212,7 +212,7 @@ describe('date changes in chart view should reflect in zoom', () => {
.type('2019-08-01');

cy.get('.icon-button.fa-list-ul')
.should('be.visible')
.should('exist')
.first()
.click();

Expand Down
2 changes: 1 addition & 1 deletion frontend/cypress/tests/codeView/codeView.cy.js
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,7 @@ describe('code view', () => {
.should('be.checked');

cy.get('.icon-button.fa-code')
.should('be.visible')
.should('exist')
.first()
.click();

Expand Down
6 changes: 3 additions & 3 deletions frontend/cypress/tests/codeView/codeView_checkFileTypes.cy.js
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ describe('check file types', () => {
it('check if all files types are visible by default', () => {
// open the code panel
cy.get('.icon-button.fa-code')
.should('be.visible')
.should('exist')
.first()
.click();

Expand All @@ -16,7 +16,7 @@ describe('check file types', () => {
it('uncheck all files types should show no files', () => {
// open the code panel
cy.get('.icon-button.fa-code')
.should('be.visible')
.should('exist')
.first()
.click();

Expand All @@ -34,7 +34,7 @@ describe('check file types', () => {
it('uncheck file type should uncheck all option and not show legend', () => {
// open the code panel
cy.get('.icon-button.fa-code')
.should('be.visible')
.should('exist')
.first()
.click();

Expand Down
18 changes: 9 additions & 9 deletions frontend/cypress/tests/codeView/codeView_filterGlob.cy.js
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ describe('filter glob', () => {
it('check filter glob radio button is clickable', () => {
// open the code panel
cy.get('.icon-button.fa-code')
.should('be.visible')
.should('exist')
.first()
.click();

Expand All @@ -18,7 +18,7 @@ describe('filter glob', () => {

it('check no filter glob input should show all file types by default', () => {
cy.get('.icon-button.fa-code')
.should('be.visible')
.should('exist')
.first()
.click();

Expand All @@ -33,7 +33,7 @@ describe('filter glob', () => {

it('check filter glob input should be in focus after click', () => {
cy.get('.icon-button.fa-code')
.should('be.visible')
.should('exist')
.first()
.click();

Expand All @@ -49,7 +49,7 @@ describe('filter glob', () => {

it('check filter glob input should have default empty value', () => {
cy.get('.icon-button.fa-code')
.should('be.visible')
.should('exist')
.first()
.click();

Expand All @@ -63,7 +63,7 @@ describe('filter glob', () => {

it('check deleting previously searched input to filter empty input by enter should show all file types', () => {
cy.get('.icon-button.fa-code')
.should('be.visible')
.should('exist')
.first()
.click();

Expand Down Expand Up @@ -93,7 +93,7 @@ describe('filter glob', () => {

it('check deleting previously searched input to filter empty input by clicking should show all file types', () => {
cy.get('.icon-button.fa-code')
.should('be.visible')
.should('exist')
.first()
.click();

Expand All @@ -120,7 +120,7 @@ describe('filter glob', () => {

it('check request to filter invalid glob by enter should not show any files', () => {
cy.get('.icon-button.fa-code')
.should('be.visible')
.should('exist')
.first()
.click();

Expand All @@ -144,7 +144,7 @@ describe('filter glob', () => {

it('check request to filter invalid glob by clicking should not show any files', () => {
cy.get('.icon-button.fa-code')
.should('be.visible')
.should('exist')
.first()
.click();

Expand All @@ -167,7 +167,7 @@ describe('filter glob', () => {

it('check filter glob should only show files with that extension', () => {
cy.get('.icon-button.fa-code')
.should('be.visible')
.should('exist')
.first()
.click();

Expand Down
10 changes: 5 additions & 5 deletions frontend/cypress/tests/codeView/codeView_hideFileDetails.cy.js
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ describe('hide all file details', () => {
it('check hide all file details hides the content of all the files', () => {
// open the code panel
cy.get('.icon-button.fa-code')
.should('be.visible')
.should('exist')
.first()
.click();

Expand All @@ -28,7 +28,7 @@ describe('hide all file details', () => {

it('check details of one file are shown, rest are hidden', () => {
cy.get('.icon-button.fa-code')
.should('be.visible')
.should('exist')
.first()
.click();

Expand Down Expand Up @@ -68,7 +68,7 @@ describe('hide all file details', () => {

it('check file can be hidden after scrolling', () => {
cy.get('.icon-button.fa-code')
.should('be.visible')
.should('exist')
.first()
.click();

Expand Down Expand Up @@ -119,7 +119,7 @@ describe('hide all file details', () => {

it('check show all file details shows the content of all the files', () => {
cy.get('.icon-button.fa-code')
.should('be.visible')
.should('exist')
.first()
.click();

Expand Down Expand Up @@ -148,7 +148,7 @@ describe('hide all file details', () => {

it('check collapsed file persists after sort', () => {
cy.get('.icon-button.fa-code')
.should('be.visible')
.should('exist')
.first()
.click();

Expand Down
18 changes: 9 additions & 9 deletions frontend/cypress/tests/codeView/codeView_mergeGroupBlame.cy.js
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ describe('merge group blame in code view', () => {
it('no author breakdown shown by default', () => {
// open the code panel
cy.get('.icon-button.fa-code')
.should('be.visible')
.should('exist')
.first()
.click();

Expand All @@ -28,7 +28,7 @@ describe('merge group blame in code view', () => {

// open the code panel
cy.get('.icon-button.fa-code')
.should('be.visible')
.should('exist')
.first()
.click();

Expand All @@ -54,7 +54,7 @@ describe('merge group blame in code view', () => {

// open the code panel
cy.get('.icon-button.fa-code')
.should('be.visible')
.should('exist')
.first()
.click();

Expand All @@ -81,7 +81,7 @@ describe('merge group blame in code view', () => {

// open the code panel
cy.get('.icon-button.fa-code')
.should('be.visible')
.should('exist')
.first()
.click();

Expand Down Expand Up @@ -109,7 +109,7 @@ describe('merge group blame in code view', () => {

// open the code panel
cy.get('.icon-button.fa-code')
.should('be.visible')
.should('exist')
.first()
.click();

Expand Down Expand Up @@ -137,7 +137,7 @@ describe('merge group blame in code view', () => {

// open the code panel
cy.get('.icon-button.fa-code')
.should('be.visible')
.should('exist')
.first()
.click();

Expand All @@ -162,7 +162,7 @@ describe('merge group blame in code view', () => {

// open the code panel
cy.get('.icon-button.fa-code')
.should('be.visible')
.should('exist')
.first()
.click();

Expand All @@ -186,7 +186,7 @@ describe('merge group blame in code view', () => {

// open the code panel
cy.get('.icon-button.fa-code')
.should('be.visible')
.should('exist')
.first()
.click();

Expand Down Expand Up @@ -219,7 +219,7 @@ describe('merge group blame in code view', () => {

// open the code panel
cy.get('.icon-button.fa-code')
.should('be.visible')
.should('exist')
.first()
.click();

Expand Down
2 changes: 1 addition & 1 deletion frontend/cypress/tests/codeView/codeView_reload.cy.js
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ describe('reload page', () => {
it('reload page should restore all controls', () => {
// open the code panel
cy.get('.icon-button.fa-code')
.should('be.visible')
.should('exist')
.first()
.click();

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -371,7 +371,7 @@ describe('render filter hash', () => {
it('code panel: sort by: url params should persist after change and reload', () => {
// open the code panel
cy.get('.icon-button.fa-code')
.should('be.visible')
.should('exist')
.first()
.click();

Expand Down Expand Up @@ -416,7 +416,7 @@ describe('render filter hash', () => {
it('code panel: order: url params should persist after change and reload', () => {
// open the code panel
cy.get('.icon-button.fa-code')
.should('be.visible')
.should('exist')
.first()
.click();

Expand Down Expand Up @@ -462,7 +462,7 @@ describe('render filter hash', () => {
it('code panel: filter by glob: url params should persist after change and reload', () => {
// open the code panel
cy.get('.icon-button.fa-code')
.should('be.visible')
.should('exist')
.first()
.click();

Expand Down
Loading
Loading