Skip to content

Rename spec "javascripts" folder to align to app "javascript"#8301

Merged
aduth merged 2 commits intomainfrom
aduth-mv-spec-javascripts
Apr 28, 2023
Merged

Rename spec "javascripts" folder to align to app "javascript"#8301
aduth merged 2 commits intomainfrom
aduth-mv-spec-javascripts

Conversation

@aduth
Copy link
Contributor

@aduth aduth commented Apr 27, 2023

🛠 Summary of changes

Renames spec/javascripts to spec/javascript, based on assumed convention that the app and spec folder structures are meant to align.

Before:

app/
  - javascript
spec/
  - javascripts

After:

app/
  - javascript
spec/
  - javascript

📜 Testing Plan

yarn test

changelog: Internal, Code Quality, Improve consistency of test folder naming
Copy link
Contributor

@zachmargolis zachmargolis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, but probably want to update package.json to only use the singular path right?

-    "test": "mocha 'spec/javascripts/**/**spec.+(j|t)s?(x)' 'app/javascript/packages/**/*.spec.+(j|t)s?(x)'",
+    "test": "mocha 'spec/javascript/**/**spec.+(j|t)s?(x)' 'app/javascript/packages/**/*.spec.+(j|t)s?(x)'",

@aduth
Copy link
Contributor Author

aduth commented Apr 27, 2023

LGTM, but probably want to update package.json to only use the singular path right?

Ah, yes indeed, that could have been bad 😅 I'll do a pass at any other references that might exist.

@aduth
Copy link
Contributor Author

aduth commented Apr 27, 2023

LGTM, but probably want to update package.json to only use the singular path right?

Ah, yes indeed, that could have been bad 😅 I'll do a pass at any other references that might exist.

Updated in c822739.

@aduth aduth merged commit 354e8ff into main Apr 28, 2023
@aduth aduth deleted the aduth-mv-spec-javascripts branch April 28, 2023 18:58
@mdiarra3 mdiarra3 mentioned this pull request May 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants