Skip to content
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

2283: Optimize CI process for tests #2516

Merged
merged 42 commits into from
Jan 15, 2025
Merged

Conversation

ShaunKav
Copy link
Collaborator

@ShaunKav ShaunKav commented Jan 2, 2025

Related Issue

Closes #2283

Description of Changes

This PR adds two new repo vars that track the last successful build's workflow id. We then use this to download the build artifact from, if there were only test changes. This will allow for quick changes to the test suite that don't require a full rebuild.

Reviewer(s)

@grenos @Doublemme @mfrezzati

UI/UX Changes

Checklist

  • Code adheres to project guidelines and conventions.
  • Unit tests, if applicable, are updated and passing.
  • Documentation, if applicable, has been updated.
  • UI/UX changes include visuals (video or screenshots).

Thank you for contributing! 🎉

Copy link

github-actions bot commented Jan 2, 2025

Jest Report

Summary

Lines Statements Branches Functions
Coverage: 92%
92.99% (1753/1885) 85.34% (530/621) 95.87% (302/315)

Tests

Tests Skipped Failures Errors Time
872 0 💤 0 ❌ 0 🔥 13m 6s ⏱️
Coverage (92%)
File% Stmts% Branch% Funcs% LinesUncovered Line #s
All files92.9985.3495.8792.75 
AccountUtils100100100100 
   AccountUtils.ts100100100100 
ActivityUtils100100100100 
   ActivityUtils.ts100100100100 
AddressUtils100100100100 
   AddressUtils.ts100100100100 
AlertUtils10010080100 
   Alert.ts10010080100 
ArrayUtils100100100100 
   ArrayUtils.ts100100100100 
BalanceUtils97.878010097.61 
   BalanceUtils.ts97.878010097.6178
BigNumberUtils10097.22100100 
   BigNumberUtils.ts10097.22100100184
BiometricsUtils95.389591.6696.29 
   BiometricErrors.ts88.8891.668091.354–55
   Biometrics.ts100100100100 
BloomUtils100100100100 
   BloomUtils.ts100100100100 
CacheKeyUtils100100100100 
   CacheKeyUtils.ts100100100100 
ColorUtils100100100100 
   ColorUtils.ts100100100100 
CryptoUtils96.5978.5710097.56 
   CryptoUtils.ts96.5978.5710097.5650, 68
CryptoUtils/Helpers76.3458.6288.8876.34 
   fastKeystoreDecrypt.ts76.3458.6288.8876.3433, 53, 69, 75, 80, 104, 110, 126, 138–162
CryptoUtils_Legacy100100100100 
   CryptoUtils_Legacy.ts100100100100 
CryptoUtils_Legacy/Helpers76.3458.6288.8876.34 
   fastKeystoreDecrypt.ts76.3458.6288.8876.3433, 53, 69, 75, 80, 104, 110, 126, 138–162
DAppUtils63.1552.837561.81 
   DAppUtils.ts63.1552.837561.8158, 65–116
DateUtils100100100100 
   DateUtils.ts100100100100 
DeviceUtils100100100100 
   DeviceUtils.ts100100100100 
ErrorMessageUtils100100100100 
   ErrorMessageUtils.ts100100100100 
FormUtils100100100100 
   FormUtils.ts100100100100 
FormattingUtils96.8791.66100100 
   FormattingUtils.ts96.8791.6610010026
GasUtils93.0573.810094.11 
   GasUtils.ts93.0573.810094.1152, 202–206
HexUtils94.2875100100 
   HexUtils.ts94.287510010063–65
IPFSUtils100100100100 
   IPFSUtils.ts100100100100 
LedgerUtils75.2491.6683.3373.95 
   LedgerUtils.ts75.2491.6683.3373.9583–87, 125–150, 179–184, 229–237, 294, 313–314
LocaleUtils10050100100 
   LocaleUtils.ts100501001006
LockScreenUtils100100100100 
   LockScreenUtils.ts100100100100 
Logger958010095 
   Logger.ts95801009538
MathUtils100100100100 
   MathUtils.ts100100100100 
MediaUtils10095.65100100 
   MediaUtils.ts10095.6510010078
MergeUtils100100100100 
   MergeUtils.ts100100100100 
PasswordUtils76.19505075 
   PasswordUtils.ts76.1950507563–64, 68, 72–73
PicassoUtils100100100100 
   PicassoUtils.ts100100100100 
PlatformUtils100100100100 
   PlatformUtils.ts100100100100 
ReanimatedUtils91.8973.3310093.93 
   Reanimated.ts91.8973.3310093.9351, 63
SemanticVersionUtils100100100100 
   SemanticVersionUtils.ts100100100100 
SignMessageUtils100100100100 
   SignMessageUtils.ts100100100100 
StringUtils100100100100 
   StringUtils.ts100100100100 
TokenUtils100100100100 
   TokenUtils.ts100100100100 
TransactionUtils97.2692.197.2997.64 
   TransactionUtils.ts97.2692.197.2997.64706–712, 802–806
TransfersUtils10080100100 
   TransfersUtils.ts10080100100100
URIUtils100100100100 
   URIUtils.ts100100100100 
WalletConnectUtils89.990.988.2390.29 
   WalletConnectUtils.tsx89.990.988.2390.2924–47

@ShaunKav ShaunKav temporarily deployed to android-release January 6, 2025 12:02 — with GitHub Actions Inactive
Copy link

socket-security bot commented Jan 14, 2025

New, updated, and removed dependencies detected. Learn more about Socket for GitHub ↗︎

Package New capabilities Transitives Size Publisher
npm/@babel/[email protected] 🔁 npm/@babel/[email protected] None +1 164 kB nicolo-ribaudo
npm/@babel/[email protected] 🔁 npm/@babel/[email protected] None 0 551 kB existentialism, hzoo, jlhwung, ...1 more
npm/@babel/[email protected] 🔁 npm/@babel/[email protected] None +1 68.4 kB existentialism, hzoo, jlhwung, ...1 more
npm/@babel/[email protected] 🔁 npm/@babel/[email protected] None 0 43.3 kB existentialism, hzoo, jlhwung, ...1 more
npm/@babel/[email protected] 🔁 npm/@babel/[email protected] None 0 7.15 kB existentialism, hzoo, jlhwung, ...1 more
npm/@babel/[email protected] 🔁 npm/@babel/[email protected] None 0 18.1 kB existentialism, hzoo, jlhwung, ...1 more
npm/@babel/[email protected] 🔁 npm/@babel/[email protected] None 0 10.4 kB existentialism, hzoo, jlhwung, ...1 more
npm/@babel/[email protected] 🔁 npm/@babel/[email protected] None 0 140 kB existentialism, hzoo, jlhwung, ...1 more
npm/@babel/[email protected] 🔁 npm/@babel/[email protected] Transitive: environment +1 706 kB nicolo-ribaudo
npm/@babel/[email protected] 🔁 npm/@babel/[email protected] None 0 2.56 MB nicolo-ribaudo
npm/@expo/[email protected] 🔁 npm/@expo/[email protected] None +1 30.7 kB alanhughes, aleqsio, betoatexpo, ...25 more
npm/@formatjs/[email protected] 🔁 npm/@formatjs/[email protected] None 0 9.19 kB longlho
npm/@formatjs/[email protected] 🔁 npm/@formatjs/[email protected] None +2 1.03 MB longlho
npm/@noble/[email protected] 🔁 npm/@noble/[email protected] None 0 729 kB paulmillr
npm/@noble/[email protected] None 0 924 kB paulmillr
npm/@tanstack/[email protected] 🔁 npm/@tanstack/[email protected] environment 0 1.95 MB tannerlinsley
npm/@tanstack/[email protected] 🔁 npm/@tanstack/[email protected] None 0 135 kB tannerlinsley
npm/@tanstack/[email protected] 🔁 npm/@tanstack/[email protected] None 0 38.4 kB tannerlinsley
npm/@tanstack/[email protected] 🔁 npm/@tanstack/[email protected] None 0 31.5 kB tannerlinsley
npm/@tanstack/[email protected] 🔁 npm/@tanstack/[email protected] None 0 715 kB nksaraf, tannerlinsley
npm/@types/[email protected] 🔁 npm/@types/[email protected] None 0 868 kB types
npm/@types/[email protected] 🔁 npm/@types/[email protected] None 0 800 kB types
npm/[email protected] 🔁 npm/[email protected] None 0 33.9 kB fengmk2
npm/[email protected] 🔁 npm/[email protected] None 0 30.8 kB yassernasc
npm/[email protected] 🔁 npm/[email protected] None 0 2.15 MB ai, beneb, caniuse-lite
npm/[email protected] 🔁 npm/[email protected] None 0 314 kB atinux, pi0
npm/[email protected] 🔁 npm/[email protected] None +1 833 kB zloirock
npm/[email protected] 🔁 npm/[email protected] None 0 40.5 kB asafkorem
npm/[email protected] 🔁 npm/[email protected] None +6 15.1 MB wix.mobile
npm/[email protected] 🔁 npm/[email protected] None 0 167 kB feedic
npm/[email protected] 🔁 npm/[email protected] None 0 154 kB kilianvalkhof
npm/[email protected] 🔁 npm/[email protected] None 0 2.39 MB ljharb
npm/[email protected] 🔁 npm/[email protected] None 0 9.88 kB ljharb
npm/[email protected] 🔁 npm/[email protected] None 0 14.5 kB ljharb
npm/[email protected] 🔁 npm/[email protected] Transitive: environment +1 1.07 MB ljharb, yannickcr
npm/[email protected] 🔁 npm/[email protected] None +1 155 kB mrmlnc
npm/[email protected] 🔁 npm/[email protected] None 0 109 kB climba03003, delvedor, eomm, ...7 more
npm/[email protected] 🔁 npm/[email protected] None 0 526 kB pi0
npm/[email protected] 🔁 npm/[email protected] None 0 22.7 kB ljharb, tunnckocore
npm/[email protected] 🔁 npm/[email protected] None 0 35.7 kB ljharb
npm/[email protected] 🔁 npm/[email protected] None 0 12.6 kB ljharb
npm/[email protected] 🔁 npm/[email protected] None +3 5.08 MB dschnurr, dschnurr-openai, jeevnayak, ...2 more
npm/[email protected] None 0 9.8 kB ljharb
npm/[email protected] 🔁 npm/[email protected] network 0 5.06 MB jmoxey
npm/[email protected] 🔁 npm/[email protected] environment 0 3.31 MB jakub.piasecki
npm/[email protected] 🔁 npm/[email protected] None +1 55.1 kB ljharb
npm/[email protected] None 0 8.5 kB ljharb
npm/[email protected] None 0 9.3 kB ljharb
npm/[email protected] 🔁 npm/[email protected], npm/[email protected] None 0 17.1 kB mafintosh, maxogden
npm/[email protected] 🔁 npm/[email protected] None 0 22.3 kB ljharb
npm/[email protected] 🔁 npm/[email protected] None 0 287 kB pi0
npm/[email protected] 🔁 npm/[email protected] None 0 14.7 kB ai
npm/[email protected] 🔁 npm/[email protected] None +6 19.7 MB awkweb, jmoxey
npm/[email protected] 🔁 npm/[email protected] None 0 681 kB eemeli

🚮 Removed packages: npm/@parcel/[email protected], npm/@parcel/[email protected], npm/@parcel/[email protected], npm/@parcel/[email protected], npm/@parcel/[email protected], npm/@parcel/[email protected], npm/@parcel/[email protected], npm/@parcel/[email protected], npm/@parcel/[email protected], npm/@parcel/[email protected], npm/@parcel/[email protected], npm/@parcel/[email protected], npm/@parcel/[email protected], npm/@parcel/[email protected], npm/@parcel/[email protected], npm/[email protected], npm/[email protected], npm/[email protected], npm/[email protected], npm/[email protected], npm/[email protected], npm/[email protected], npm/[email protected], npm/[email protected], npm/[email protected], npm/[email protected], npm/[email protected], npm/[email protected], npm/[email protected], npm/[email protected], npm/[email protected], npm/[email protected], npm/[email protected], npm/[email protected], npm/[email protected]

View full report↗︎

@Doublemme Doublemme marked this pull request as ready for review January 15, 2025 08:45
@Doublemme Doublemme merged commit ba5b32a into main Jan 15, 2025
8 checks passed
@Doublemme Doublemme deleted the 2283-optimize-ci-process-for-tests branch January 15, 2025 08:48
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.

Optimize CI Process for Maestro Test Script Changes
2 participants