chore(config): add reference config auditing

a11ywatch · Jun 26, 2024 · 9e98a49 · 9e98a49
1 parent 8af7c56
commit 9e98a49
Show file tree

Hide file tree

Showing 22 changed files with 120 additions and 83 deletions.
diff --git a/Cargo.lock b/Cargo.lock
diff --git a/Cargo.toml b/Cargo.toml
@@ -12,6 +12,6 @@ members = [
 smallvec = "1"
 selectors = "0.22.0"
 cssparser = "0.27.0"
-fast_html5ever = "0.26.1"
+fast_html5ever = "0.26.6"
 ego-tree = "0.6.2"
 lazy_static = "1.4"
diff --git a/README.md b/README.md
@@ -6,7 +6,7 @@ The Rust web accessibility engine.
 
 ```toml
 [dependencies]
-accessibility-rs = "^0.0.64"
+accessibility-rs = "^0.1"
 ```
 
 ```rs
@@ -32,7 +32,7 @@ With the Tokio runtime.
 
 ```toml
 [dependencies]
-accessibility-rs = { version = "^0.0.64", features = ["tokio"]}
+accessibility-rs = { version = "^0.1", features = ["tokio"]}
 ```
 
 ```rs
@@ -60,7 +60,7 @@ With the Spider full website crawling.
 
 ```toml
 [dependencies]
-accessibility-rs = { version = "^0.0.64", features = ["spider"]}
+accessibility-rs = { version = "^0.1", features = ["spider"]}
 ```
 
 ```rs
@@ -74,6 +74,28 @@ async fn main() {
 }
 ```
 
+If you need to use concurrency use TendrilSink.
+
+```rs
+use accessibility_rs::{fast_html5ever, Auditor, Html};
+use fast_html5ever::driver::{self, ParseOpts};
+use tendril::TendrilSink;
+
+let parser = driver::parse_document(
+    Html::new_document(),
+    ParseOpts::default(),
+);
+let document = parser.one("<html>MY html code </html>");
+
+let auditor = Auditor::new(
+    &document, &"", false, &"en",
+);
+
+let issues =
+    accessibility_rs::engine::audit::wcag::WCAGAAA::audit(auditor)
+        .await;
+```
+
 ### Documentation
 
 [Module documentation with examples](https://docs.rs/accessibility-rs).
@@ -87,6 +109,7 @@ async fn main() {
 1. i18n support for multiple languages.
 1. Re-creating layout tree to get element position coordinates.
 1. Crawling full websites lightning-fast using [spider](https://github.com/spider-rs/spider).
+1. Low-level built to be used as an engine in browsers.
 
 ## [Benchmarks](./benches/)
 

diff --git a/accessibility-rs/Cargo.toml b/accessibility-rs/Cargo.toml
@@ -1,6 +1,6 @@
 [package]
 name = "accessibility-rs"
-version = "0.0.64"
+version = "0.1.4"
 authors = ["The A11yWatch Project Developers", "Jeff Mendez <jeff@a11ywatch.com>"]
 edition = "2021"
 license = "MIT OR Apache-2.0"
@@ -12,11 +12,11 @@ include = ["/src", "../LICENSE_MIT", "../LICENSE_APACHE", "../README.md", "local
 
 [dependencies]
 lazy_static = { workspace = true }
-accessibility-scraper = { version = "0.0.12", features = ["main"], default-features = false, path = "../accessibility-scraper" }
-accessibility-tree = { version = "0.0.12", path = "../accessibility-tree/victor" }
+accessibility-scraper = { version = "0.0.14", features = ["main"], default-features = false, path = "../accessibility-scraper" }
+accessibility-tree = { version = "0.0.14", path = "../accessibility-tree/victor" }
 getrandom = { version = "0.2", features = ["js"] }
-taffy = { version = "0.4.0" }
-serde = { version = "1.0", features = ["derive"] }
+taffy = { version = "0.5" }
+serde = { version = "1", features = ["derive"] }
 selectors = { workspace = true }
 smallvec = { workspace = true }
 ego-tree = { workspace = true }
@@ -29,7 +29,7 @@ contrast = "0.1"
 rgb = "0.8"
 rayon = { version = "1", optional = true }
 crossbeam-channel = { version = "0.5", optional = true }
-tokio = { version = "1", features = ["macros"], optional = true }
+tokio = { version = "1", features = ["macros", "rt-multi-thread", "parking_lot"], optional = true }
 tokio-stream = { version = "0.1", optional = true }
 spider = { version = "1.98.2", optional = true }
 

diff --git a/accessibility-rs/src/engine/rules/rule.rs b/accessibility-rs/src/engine/rules/rule.rs
@@ -1,6 +1,6 @@
 use crate::engine::rules::techniques::Techniques;
 use crate::engine::rules::wcag_base::{Guideline, IssueType, Principle};
-use crate::ElementRef;
+use accessibility_scraper::ElementRef;
 
 /// the validation response
 #[derive(Default, Debug)]

diff --git a/accessibility-rs/src/engine/rules/utils/nodes.rs b/accessibility-rs/src/engine/rules/utils/nodes.rs
@@ -1,5 +1,5 @@
 use crate::engine::rules::rule::Validation;
-use crate::ElementRef;
+use accessibility_scraper::ElementRef;
 use accessibility_scraper::Selector;
 use selectors::Element;
 

diff --git a/accessibility-rs/src/integration_test.rs b/accessibility-rs/src/integration_test.rs
@@ -6,7 +6,7 @@ use mocks::mock;
 
 #[test]
 fn _audit() {
-    let report = accessibility_rs::audit(AuditConfig::new(
+    let report = accessibility_rs::audit(&AuditConfig::new(
         mock::MOCK_WEBSITE_HTML,
         &mock::MOCK_CSS_RULES,
         false,
@@ -17,7 +17,7 @@ fn _audit() {
 
 #[test]
 fn _audit_large() {
-    let report = accessibility_rs::audit(AuditConfig::new(
+    let report = accessibility_rs::audit(&AuditConfig::new(
         mock::MOCK_HTML_LARGE_PAGE,
         &mock::MOCK_CSS_RULES_LARGE,
         false,
@@ -28,7 +28,7 @@ fn _audit_large() {
 
 #[test]
 fn _audit_bounded() {
-    let report = accessibility_rs::audit(AuditConfig::new(
+    let report = accessibility_rs::audit(&AuditConfig::new(
         mock::MOCK_WEBSITE_HTML,
         &mock::MOCK_CSS_RULES,
         true,
@@ -39,7 +39,7 @@ fn _audit_bounded() {
 
 #[test]
 fn _audit_large_bounded() {
-    let report = accessibility_rs::audit(AuditConfig::new(
+    let report = accessibility_rs::audit(&AuditConfig::new(
         mock::MOCK_HTML_LARGE_PAGE,
         &mock::MOCK_CSS_RULES_LARGE,
         true,

diff --git a/accessibility-rs/src/lib.rs b/accessibility-rs/src/lib.rs
@@ -62,9 +62,12 @@ pub mod engine;
 /// locales for translations.
 pub mod i18n;
 
-use crate::engine::audit::auditor::Auditor;
-use crate::engine::issue::Issue;
-use accessibility_scraper::ElementRef;
+pub use accessibility_scraper;
+pub use accessibility_scraper::fast_html5ever;
+pub use accessibility_scraper::Html;
+
+pub use crate::engine::audit::auditor::Auditor;
+pub use crate::engine::issue::Issue;
 
 i18n!("locales", fallback = "en");
 
@@ -253,7 +256,7 @@ pub async fn audit(config: AuditConfig) -> AuditResults {
 
 /// audit a web page passing the html and css rules.
 #[cfg(not(feature = "tokio"))]
-pub fn audit(config: AuditConfig) -> Vec<Issue> {
+pub fn audit(config: &AuditConfig) -> Vec<Issue> {
     let document = accessibility_scraper::Html::parse_document(&config.html);
     let auditor = Auditor::new(&document, &config.css, config.bounding_box, &config.locale);
     engine::audit::wcag::WCAGAAA::audit(auditor)

diff --git a/accessibility-rs/tests/integration_test.rs b/accessibility-rs/tests/integration_test.rs
@@ -7,7 +7,7 @@ use mocks::mock;
 #[cfg(feature = "tokio")]
 #[tokio::test]
 async fn _audit() {
-    let report = accessibility_rs::audit(AuditConfig::new(
+    let report = accessibility_rs::audit(&AuditConfig::new(
         mock::MOCK_WEBSITE_HTML,
         &mock::MOCK_CSS_RULES,
         false,
@@ -20,7 +20,7 @@ async fn _audit() {
 #[test]
 #[cfg(not(feature = "tokio"))]
 fn _audit() {
-    let report = accessibility_rs::audit(AuditConfig::new(
+    let report = accessibility_rs::audit(&AuditConfig::new(
         mock::MOCK_WEBSITE_HTML,
         &mock::MOCK_CSS_RULES,
         false,
@@ -32,7 +32,7 @@ fn _audit() {
 #[test]
 #[cfg(not(feature = "tokio"))]
 fn _audit_large() {
-    let report = accessibility_rs::audit(AuditConfig::new(
+    let report = accessibility_rs::audit(&AuditConfig::new(
         mock::MOCK_HTML_LARGE_PAGE,
         &mock::MOCK_CSS_RULES_LARGE,
         false,
@@ -44,7 +44,7 @@ fn _audit_large() {
 #[test]
 #[cfg(not(feature = "tokio"))]
 fn _audit_with_layout() {
-    let report = accessibility_rs::audit(AuditConfig::new(
+    let report = accessibility_rs::audit(&AuditConfig::new(
         mock::MOCK_WEBSITE_HTML,
         &mock::MOCK_CSS_RULES,
         true,
@@ -56,7 +56,7 @@ fn _audit_with_layout() {
 #[test]
 #[cfg(not(feature = "tokio"))]
 fn _audit_large_with_layout() {
-    let report = accessibility_rs::audit(AuditConfig::new(
+    let report = accessibility_rs::audit(&AuditConfig::new(
         mock::MOCK_HTML_LARGE_PAGE,
         &mock::MOCK_CSS_RULES_LARGE,
         true,
@@ -68,7 +68,7 @@ fn _audit_large_with_layout() {
 #[test]
 #[cfg(not(feature = "tokio"))]
 fn _audit_xlarge() {
-    let report = accessibility_rs::audit(AuditConfig::new(
+    let report = accessibility_rs::audit(&AuditConfig::new(
         mock::MOCK_WEBSITE_XLARGE_HTML,
         &mock::MOCK_CSS_RULES_XLARGE,
         false,
@@ -80,7 +80,7 @@ fn _audit_xlarge() {
 #[tokio::test]
 #[cfg(feature = "tokio")]
 async fn _audit_xlarge() {
-    let report = accessibility_rs::audit(AuditConfig::new(
+    let report = accessibility_rs::audit(&AuditConfig::new(
         mock::MOCK_WEBSITE_XLARGE_HTML,
         &mock::MOCK_CSS_RULES_XLARGE,
         false,
@@ -94,6 +94,6 @@ async fn _audit_xlarge() {
 #[tokio::test]
 async fn _audit_website() {
     let audit_config = AuditConfig::new_website("https://choosealicense.com", "", false, "");
-    let report = accessibility_rs::audit(audit_config).await;
+    let report = accessibility_rs::audit(&audit_config).await;
     println!("{:?}", report)
 }
diff --git a/accessibility-rs/tests/unit/anchor.rs b/accessibility-rs/tests/unit/anchor.rs
@@ -7,7 +7,7 @@ use maud::html;
 #[cfg(not(feature = "tokio"))]
 /// anchor contains single img element without alt
 fn _audit_missing_alt_anchor_img() {
-    let audit = accessibility_rs::audit(AuditConfig::basic(
+    let audit = accessibility_rs::audit(&AuditConfig::basic(
         r###"<html xmlns="http://www.w3.org/1999/xhtml" lang="en">
     <head>     
        <title>Decrative Img: Do not use!</title>
@@ -33,7 +33,7 @@ fn _audit_missing_anchor_content_valid_href() {
     let markup = html! {
         a href="www.example.com";
     };
-    let audit = accessibility_rs::audit(AuditConfig::basic(&markup.into_string()));
+    let audit = accessibility_rs::audit(&AuditConfig::basic(&markup.into_string()));
     let valid = !audit
         .iter()
         .any(|x| x.code == "WCAGAAA.Principle4.Guideline4_1.H91");
@@ -48,7 +48,7 @@ fn _audit_missing_anchor_content() {
     let markup = html! {
         a { "" }
     };
-    let audit = accessibility_rs::audit(AuditConfig::basic(&markup.into_string()));
+    let audit = accessibility_rs::audit(&AuditConfig::basic(&markup.into_string()));
     let valid = !audit
         .iter()
         .any(|x| x.code == "WCAGAAA.Principle4.Guideline4_1.H91");
@@ -60,7 +60,7 @@ fn _audit_missing_anchor_content() {
 #[cfg(not(feature = "tokio"))]
 /// anchor text matches img alt
 fn _audit_img_alt_matches_text_anchor() {
-    let audit = accessibility_rs::audit(AuditConfig::basic(
+    let audit = accessibility_rs::audit(&AuditConfig::basic(
         r###"<html xmlns="http://www.w3.org/1999/xhtml" lang="en">
     <head>     
        <title>Decrative Img: Do not use!</title>

diff --git a/accessibility-rs/tests/unit/applet.rs b/accessibility-rs/tests/unit/applet.rs
@@ -6,7 +6,7 @@ use accessibility_rs::AuditConfig;
 #[cfg(not(feature = "tokio"))]
 /// missing applet alt
 fn _audit_missing_applet_alt() {
-    let audit = accessibility_rs::audit(AuditConfig::basic(
+    let audit = accessibility_rs::audit(&AuditConfig::basic(
         r###"<html lang="en">
        <head>     
           <title>Missing applet alt: Do not use this!</title>
@@ -34,7 +34,7 @@ fn _audit_missing_applet_alt() {
 #[cfg(not(feature = "tokio"))]
 /// missing applet body
 fn _audit_missing_applet_body() {
-    let audit = accessibility_rs::audit(AuditConfig::basic(
+    let audit = accessibility_rs::audit(&AuditConfig::basic(
         r###"<html lang="en">
        <head>     
           <title>Missing applet alt: Do not use this!</title>

diff --git a/accessibility-rs/tests/unit/area.rs b/accessibility-rs/tests/unit/area.rs
@@ -14,7 +14,7 @@ fn _audit_missing_alt_area() {
       <area shape="circle" coords="337,300,44" alt="Cup of coffee" href="coffee.htm">
     </map>"#;
     let audit: Vec<accessibility_rs::engine::issue::Issue> =
-        accessibility_rs::audit(AuditConfig::basic(html));
+        accessibility_rs::audit(&AuditConfig::basic(html));
 
     let valid = audit
         .iter()
@@ -35,7 +35,7 @@ fn _audit_missing_alt_map() {
       <area shape="circle" coords="337,300,44" alt="Cup of coffee" href="coffee.htm">
     </map>"#;
     let audit: Vec<accessibility_rs::engine::issue::Issue> =
-        accessibility_rs::audit(AuditConfig::basic(html));
+        accessibility_rs::audit(&AuditConfig::basic(html));
 
     let valid = audit.iter().any(|x| x.message == "1_1_1_H24.ImageMapNoAlt");
 

diff --git a/accessibility-rs/tests/unit/contrast.rs b/accessibility-rs/tests/unit/contrast.rs
@@ -12,7 +12,7 @@ fn _audit_contrast_text_anchor() {
             a href="www.example.com" style="color:blue;" { "example" };
         }
     };
-    let audit = accessibility_rs::audit(AuditConfig::basic(&markup.into_string()));
+    let audit = accessibility_rs::audit(&AuditConfig::basic(&markup.into_string()));
 
     let valid = !audit
         .iter()

diff --git a/accessibility-rs/tests/unit/fieldset.rs b/accessibility-rs/tests/unit/fieldset.rs
@@ -15,7 +15,7 @@ fn _audit_missing_fieldset_legend() {
         }
     };
 
-    let audit = accessibility_rs::audit(AuditConfig::basic(&m.into_string()));
+    let audit = accessibility_rs::audit(&AuditConfig::basic(&m.into_string()));
 
     let valid = !audit
         .iter()
-Original file line number
+Diff line change
@@ Expand Up / @@ -15,7 +15,7 @@ fn _audit_missing_fieldset_legend() { @@
             }
         };
-        let audit = accessibility_rs::audit(AuditConfig::basic(&m.into_string()));
+        let audit = accessibility_rs::audit(&AuditConfig::basic(&m.into_string()));
         let valid = !audit
             .iter()
@@ Expand Down @@