Skip to content

Add tests for 'notifications' reducer (#2082) #5374

Add tests for 'notifications' reducer (#2082)

Add tests for 'notifications' reducer (#2082) #5374

Triggered via pull request January 11, 2025 02:11
Status Failure
Total duration 2m 22s
Artifacts

ci.yml

on: pull_request

Annotations

10 errors and 11 warnings
build: test/reducers/notifications.spec.js#L6
Split 'let' declarations into multiple statements
build: test/reducers/notifications.spec.js#L6
Expected variable declaration to be on a new line
build: test/reducers/notifications.spec.js#L6
Expected variable declaration to be on a new line
build: test/reducers/notifications.spec.js#L6
Expected variable declaration to be on a new line
build: test/reducers/notifications.spec.js#L55
Trailing spaces not allowed
build: test/reducers/notifications.spec.js#L57
Missing semicolon
build: test/reducers/notifications.spec.js#L62
Missing semicolon
build: test/reducers/notifications.spec.js#L78
Trailing spaces not allowed
build: test/reducers/notifications.spec.js#L96
Extra space before value for key 'message'
build: test/reducers/notifications.spec.js#L97
Missing semicolon
build
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
build: app/assets/javascripts/components/common/ArticleViewer/components/TitleOpener.jsx#L7
Use I18n over string literals for localization: <p id="icon-article-viewer-desc">Open Article Viewer</p>
build: app/assets/javascripts/components/common/AssignmentLinks/AllPeerReviewLinks.jsx#L10
Use I18n over string literals for localization: <span key="title">Reviews: </span>
build: app/assets/javascripts/components/common/enroll_button.jsx#L132
Use I18n over string literals for localization: <a href={massEnrollmentUrl}>Add multiple users at once.</a>
build: app/assets/javascripts/components/common/final_article_checklist.jsx#L41
Use I18n over string literals for localization: <button onClick={hide} className="button dark small">Okay</button>
build: app/assets/javascripts/components/common/final_article_checklist.jsx#L43
Use I18n over string literals for localization: <a onClick={show} className="button dark small">Quality checklist</a>
build: app/assets/javascripts/components/common/final_article_checklist.jsx#L52
Use I18n over string literals for localization: <h2>Final review checklist</h2>
build: app/assets/javascripts/components/common/get_help_button.jsx#L114
Use I18n over string literals for localization: <a href="#" className="wikipedia-help-link button dark small stacked" onClick={e => this.updateTargetUser(wikipediaHelpUser, e)}>question about editing Wikipedia</a>
build: app/assets/javascripts/components/common/get_help_button.jsx#L124
Use I18n over string literals for localization: <a href="#" className="program-help-link button dark stacked small" onClick={e => this.updateTargetUser(programHelpUser, e)}>question about Wiki Ed or your assignment</a>
build: app/assets/javascripts/components/common/get_help_button.jsx#L134
Use I18n over string literals for localization: <a href="#" className="dashboard-help-link button dark stacked small" onClick={e => this.updateTargetUser(dashboardHelpUser, e)}>question about the dashboard</a>
build: app/assets/javascripts/components/common/mainspace_checklist.jsx#L20
Use I18n over string literals for localization: <button onClick={hide} className="button dark small">Okay</button>