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

Port the tests from the B&N contribution #1146

Closed
rdeltour opened this issue May 28, 2020 · 0 comments
Closed

Port the tests from the B&N contribution #1146

rdeltour opened this issue May 28, 2020 · 0 comments
Assignees
Labels
type: tests The issue is related to the test suite

Comments

@rdeltour
Copy link
Member

This is a placeholder issue to keep track of the work in the milestone.

@rdeltour rdeltour added the type: tests The issue is related to the test suite label May 28, 2020
@rdeltour rdeltour added this to the Test Suite Cleanup milestone May 28, 2020
@rdeltour rdeltour self-assigned this May 28, 2020
rdeltour added a commit that referenced this issue Jul 13, 2020
These tests were initially contributed by B&N for EPUBCheck v4.0. They
were very brittle: each test was effectively comparing the entire output
details (all messages + EPUB analytics) with expected results.

This commit ports the functional tests in the `com.adobe.epubcheck.test`
package the new BDD framework. Only the tests for ERROR and WARNING
messages are ported. Many USAGE messages originally introduced by the
same contribution are not very useful. The tests for these have been
discarded.

This commit also:
- moves the `Messages` localization test to `LocalizedMessagesTest.java`
  in the messages package. These tests will have to be improved when
  we refactor the messaging system.
- remove the `ApiConstructorsTest` (not very extensive). API testing will
  be improved when we refactor the public API.
- remove the coverage test, which was brittle and not very meaningful.

Close #1146.
rdeltour added a commit that referenced this issue Jul 8, 2022
---------------------

docs: switch to higher quality reference for alternative text

Co-authored-by: Jens Oliver Meiert <jens@meiert.com>

---------------------

switch to higher quality reference for alternative text

fixes #1146

Co-authored-by: Steve Faulkner <faulkner.steve@gmail.com>

---------------------

ARIA: Allow aria-required with special input types

Add aria-required to input types datetime-local / date / month / time / week

Co-authored-by: Daniel <revoltek-daniel@users.noreply.github.com>

---------------------

Allow rel attribute on SVG <a> element

Fixes validator/validator#1008
Closes validator/validator#1009

Co-authored-by: MattGarrish <mattgarrish@users.noreply.github.com>

---------------------

Allow any ARIA role for <svg> element

Fixes validator/validator#1096
Fixes validator/validator#981

Co-authored-by: Michael[tm] Smith <mike@w3.org>

---------------------

Allow aria.prop.posinset|setsize w/ role=tab

Fixes validator/validator#1060

Co-authored-by: Michael[tm] Smith <mike@w3.org>

---------------------

Allow “aria-placeholder” where “placeholder” is allowed

Also, report error if element with “placeholder” also has
“aria-placeholder”.

Fixes validator/validator#1098

Co-authored-by: Michael[tm] Smith <mike@w3.org>

---------------------

Allow only presentation|none for br[role]

Fixes validator/validator#1097

Co-authored-by: Michael[tm] Smith <mike@w3.org>

---------------------

Update required states for input[type=radio|checkbox]

Fixes validator/validator#1109

Co-authored-by: Michael[tm] Smith <mike@w3.org>

---------------------

Update allowed roles for figure element

Fixes validator/validator#1108

Co-authored-by: Michael[tm] Smith <mike@w3.org>

---------------------

Allow th[aria-rowspan] & th[aria-colspan]

Relates to validator/validator#1107

Co-authored-by: Michael[tm] Smith <mike@w3.org>

---------------------

Allow math element to have global ARIA attributes

Fixes validator/validator#1114

Co-authored-by: Michael[tm] Smith <mike@w3.org>

---------------------

Add menu, menubar, and tablist to allowed nav roles

Fixes validator/validator#1110

Co-authored-by: Michael[tm] Smith <mike@w3.org>

---------------------

Align aria-valuemin checking with current spec

Fixes validator/validator#1120

Co-authored-by: Michael[tm] Smith <mike@w3.org>

---------------------

Align aria-valuemax checking with current spec

Fixes validator/validator#1119

Co-authored-by: Michael[tm] Smith <mike@w3.org>

---------------------

Allow aria-description

Fixes validator/validator#1155

Co-authored-by: Michael[tm] Smith <mike@w3.org>

---------------------

Allow -1 for aria-setsize

Fixes validator/validator#1158

Co-authored-by: Michael[tm] Smith <mike@w3.org>

---------------------

update the allowed roles for wbr

update allowed roles for `wbr` to match `br` element

Co-authored-by: Scott O'Hara <scottaohara@users.noreply.github.com>
rdeltour added a commit that referenced this issue Aug 18, 2022
---------------------

docs: switch to higher quality reference for alternative text

Co-authored-by: Jens Oliver Meiert <jens@meiert.com>

---------------------

switch to higher quality reference for alternative text

fixes #1146

Co-authored-by: Steve Faulkner <faulkner.steve@gmail.com>

---------------------

ARIA: Allow aria-required with special input types

Add aria-required to input types datetime-local / date / month / time / week

Co-authored-by: Daniel <revoltek-daniel@users.noreply.github.com>

---------------------

Allow rel attribute on SVG <a> element

Fixes validator/validator#1008
Closes validator/validator#1009

Co-authored-by: MattGarrish <mattgarrish@users.noreply.github.com>

---------------------

Allow any ARIA role for <svg> element

Fixes validator/validator#1096
Fixes validator/validator#981

Co-authored-by: Michael[tm] Smith <mike@w3.org>

---------------------

Allow aria.prop.posinset|setsize w/ role=tab

Fixes validator/validator#1060

Co-authored-by: Michael[tm] Smith <mike@w3.org>

---------------------

Allow “aria-placeholder” where “placeholder” is allowed

Also, report error if element with “placeholder” also has
“aria-placeholder”.

Fixes validator/validator#1098

Co-authored-by: Michael[tm] Smith <mike@w3.org>

---------------------

Allow only presentation|none for br[role]

Fixes validator/validator#1097

Co-authored-by: Michael[tm] Smith <mike@w3.org>

---------------------

Update required states for input[type=radio|checkbox]

Fixes validator/validator#1109

Co-authored-by: Michael[tm] Smith <mike@w3.org>

---------------------

Update allowed roles for figure element

Fixes validator/validator#1108

Co-authored-by: Michael[tm] Smith <mike@w3.org>

---------------------

Allow th[aria-rowspan] & th[aria-colspan]

Relates to validator/validator#1107

Co-authored-by: Michael[tm] Smith <mike@w3.org>

---------------------

Allow math element to have global ARIA attributes

Fixes validator/validator#1114

Co-authored-by: Michael[tm] Smith <mike@w3.org>

---------------------

Add menu, menubar, and tablist to allowed nav roles

Fixes validator/validator#1110

Co-authored-by: Michael[tm] Smith <mike@w3.org>

---------------------

Align aria-valuemin checking with current spec

Fixes validator/validator#1120

Co-authored-by: Michael[tm] Smith <mike@w3.org>

---------------------

Align aria-valuemax checking with current spec

Fixes validator/validator#1119

Co-authored-by: Michael[tm] Smith <mike@w3.org>

---------------------

Allow aria-description

Fixes validator/validator#1155

Co-authored-by: Michael[tm] Smith <mike@w3.org>

---------------------

Allow -1 for aria-setsize

Fixes validator/validator#1158

Co-authored-by: Michael[tm] Smith <mike@w3.org>

---------------------

update the allowed roles for wbr

update allowed roles for `wbr` to match `br` element

Co-authored-by: Scott O'Hara <scottaohara@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: tests The issue is related to the test suite
Projects
None yet
Development

No branches or pull requests

1 participant