-
Notifications
You must be signed in to change notification settings - Fork 2.9k
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
[No QA] [TS migration] Migrate 'E2E' lib to TypeScript #28896
Merged
Merged
Changes from all commits
Commits
Show all changes
26 commits
Select commit
Hold shift + click to select a range
65c2205
[TS migration] Migrate 'E2E' lib
VickyStash cbf918c
Merge branch 'main' into ts-migration/e2e-lib
VickyStash 61aa3d8
Add errors field to report action type
VickyStash ba6802d
Merge branch 'main' into ts-migration/e2e-lib
VickyStash 354cd4c
Migrate e2e files to TS
VickyStash 82fb834
Merge branch 'main' into ts-migration/e2e-lib
VickyStash af68ede
Update files extension in docs
VickyStash 8b9f6ff
Merge branch 'main' into ts-migration/e2e-lib
VickyStash bc08d2a
Remove todo
VickyStash ed0bfd3
Merge branch 'main' into ts-migration/e2e-lib
VickyStash 21bf425
Fix ts error
VickyStash 61c5075
Merge branch 'main' into ts-migration/e2e-lib
VickyStash 551e96a
Update imports after merging main, fix ts errors
VickyStash 5bf3423
Merge branch 'main' into ts-migration/e2e-lib
VickyStash 3e0627a
Fix prettier
VickyStash e912bc2
Merge branch 'main' into ts-migration/e2e-lib
VickyStash 1eaac4c
Fix NaN in cooling down logs
VickyStash a3361ca
Merge branch 'main' into ts-migration/e2e-lib
VickyStash 581dfcd
Migrate new files to TS; fix TS issues after merging main
VickyStash fe890c9
Merge branch 'main' into ts-migration/e2e-lib
VickyStash 7d0075e
Fix ts issues after main merging
VickyStash 876fc14
Update file extension
VickyStash f7763cf
Merge branch 'main' into ts-migration/e2e-lib
VickyStash a5d2248
Remove extra params from Report type
VickyStash 4479acc
Merge branch 'main' into ts-migration/e2e-lib
VickyStash d790b48
Fix ts issues
VickyStash File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
2 changes: 1 addition & 1 deletion
2
src/libs/E2E/actions/waitForKeyboard.js → src/libs/E2E/actions/waitForKeyboard.ts
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file was deleted.
Oops, something went wrong.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,11 @@ | ||
import Response from '@src/types/onyx/Response'; | ||
|
||
const authenticatePusher = (): Response => ({ | ||
auth: 'auth', | ||
// eslint-disable-next-line @typescript-eslint/naming-convention | ||
shared_secret: 'secret', | ||
jsonCode: 200, | ||
requestID: '783ef7fc3991969a-SJC', | ||
}); | ||
|
||
export default authenticatePusher; |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you give a bit of context here? What is the source of all these new properties?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@blazejkustra CONST.BETAS is used in Beta type
There are two apiMocks that set betas, inside openApp.ts and signinUser.ts
So without these new properties, it gives an error like this:
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I am working on removing all useless ones, and this change made it 10x harder in this repo 😞
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@flodnv Why it made harder for you? Can you give a bit more context?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Because now I have to check if all of these betas are used, which they probably are not...
It would have been better to update the mocks to match the code instead of the opposite, which I think is probably almost always preferred (why mock something that does not exist / is not used?)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed here: #31783