docs(claude-md): strengthen testing requirements and add UI test command#1465
docs(claude-md): strengthen testing requirements and add UI test command#1465J2TeamNNL wants to merge 4 commits into
Conversation
|
Thank you for your contribution! Before we can merge this PR, you need to sign our Contributor License Agreement. To sign, please comment below with:
I have read the CLA Document and I hereby sign the CLA. 1 out of 2 committers have signed the CLA. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 7271232286
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
| xcodebuild -project TablePro.xcodeproj -scheme TablePro test -skipPackagePluginValidation | ||
| xcodebuild -project TablePro.xcodeproj -scheme TablePro test -skipPackagePluginValidation -only-testing:TableProTests/TestClassName | ||
| xcodebuild -project TablePro.xcodeproj -scheme TablePro test -skipPackagePluginValidation -only-testing:TableProTests/TestClassName/testMethodName | ||
| xcodebuild -project TablePro.xcodeproj -scheme TablePro test -skipPackagePluginValidation -only-testing:TableProUITests |
There was a problem hiding this comment.
Point the UI test command at an existing target
In this repo the new command selects TableProUITests, but I inspected TablePro.xcodeproj/project.pbxproj and TablePro.xcodeproj/xcshareddata/xcschemes/TablePro.xcscheme, and the only test target/testable is TableProTests. Apple documents -only-testing identifiers as being rooted at the test target (test_target/test_type/test_function), so this command cannot run a valid bundle here and will fail for anyone following the added UI-test guidance until a TableProUITests target is added or the command is removed.
Useful? React with 👍 / 👎.
Summary
xcodebuild ... -only-testing:TableProUITestscommand to the Build & Development Commands sectionThese are project-convention changes that affect every future PR.