fix: Adjust stylelint rules and fix E2E tests

Appsilon · Jan 5, 2024 · b59692d · b59692d
1 parent e4fefc6
commit b59692d
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 3 deletions.
diff --git a/inst/templates/node/dot.stylelintrc.json b/inst/templates/node/dot.stylelintrc.json
@@ -3,6 +3,12 @@
   "rules": {
     "at-rule-no-unknown": null,
     "no-empty-source": null,
-    "scss/at-rule-no-unknown": true
+    "scss/at-rule-no-unknown": true,
+    "selector-id-pattern": [
+      "^([a-z][a-z0-9]*)([-_][a-z0-9]+)*$",
+      {
+        "message": "Expected ID selector to be kebab-snake_case"
+      }
+    ]
   }
 }
diff --git a/tests/e2e/test-lint-sass.R b/tests/e2e/test-lint-sass.R
@@ -2,12 +2,12 @@ rhino::lint_sass()
 
 # Create bad scripts and test if formatting returns the expected result
 test_scss_path <- fs::path("app", "styles", "bad-style.scss")
-cat(".myClass{color: #FFFFFF}", file = test_scss_path)
+cat(".my-class{color: #FFFFFF}", file = test_scss_path)
 testthat::expect_error(rhino::lint_sass())
 rhino::lint_sass(fix = TRUE)
 testthat::expect_identical(
   readLines(test_scss_path),
-  ".myClass { color: #fff; }"
+  ".my-class { color: #fff; }"
 )
 # Clean up
 file.remove(test_scss_path)