-
Notifications
You must be signed in to change notification settings - Fork 591
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
build: make tests run faster #4205
Merged
Merged
Changes from 96 commits
Commits
Show all changes
98 commits
Select commit
Hold shift + click to select a range
138e231
attempt1
sofisl 0a4110a
retry
sofisl e3aede8
retry
sofisl 2852e1e
retry
sofisl ee931df
retry
sofisl 11ea3dc
retry
sofisl b3d5f5a
retry
sofisl e334f94
delete old jobs
sofisl e585fc1
retry
sofisl ad992e2
Update deletecloudbuild.sh
sofisl 2dcaec5
merge conflict
sofisl 0324464
retry dockerfile
sofisl f5a30cc
update single test
sofisl d621b15
use curl to install pnpm
sofisl 4222139
retry
sofisl 39cc661
retry
sofisl 77d53f2
retry
sofisl 8f07616
retry
sofisl 72b8e50
retry
sofisl c560514
which pnpm
sofisl e586ef4
retry
sofisl 1706f3a
retry
sofisl fac653b
retry
sofisl 9c5ceff
retry
sofisl 3eb9a94
retry
sofisl 64ab9b8
retry
sofisl 0439945
retry
sofisl 37eca92
retry
sofisl d4a0f34
retry
sofisl c15c2d6
retry
sofisl f9493f7
retry
sofisl 2e9671c
retry
sofisl ba72dd6
retry
sofisl c4871c0
retry
sofisl 09ce8be
retry
sofisl b25218c
retry
sofisl 1b14c6b
retry
sofisl c2c607f
retry
sofisl a620295
retry
sofisl 70c7500
retry
sofisl 32889c2
retry
sofisl e2cd3f4
update package.json
sofisl 20a31e0
retretry
sofisl 6ea9768
retry
sofisl b51c842
retry
sofisl 8a3c36d
retry
sofisl 5d31751
retry
sofisl 5773d2b
retry
sofisl ab8dcb1
retry
sofisl a5097c9
retry
sofisl 9159e18
retry
sofisl 28f3766
retry
sofisl ae04c8e
mretry
sofisl b59fdbf
retry
sofisl 27bba5f
retry
sofisl 56a86c6
retry
sofisl 26c4946
remove lockfile
sofisl 496ae1b
revert change to tsconfig
sofisl aa876d0
retry
sofisl 507a1cd
retry
sofisl dcb17f9
retry
sofisl c485713
retry
sofisl 07f8157
retry
sofisl c46717a
retry
sofisl 73b7c35
retry
sofisl dcab75d
retry
sofisl 0c6d3c5
retry
sofisl 044835c
retry
sofisl 893fa57
retry
sofisl 2df831d
retry
sofisl 72153c5
changes to all package.json
sofisl da84cb7
undo changes to gapic-node-templating
sofisl ae0845b
undo changes to protos
sofisl 3c0c1cb
modify conditional tests
sofisl 0720375
retry
sofisl 3dc3236
undo changes to tsconfig
sofisl 0983ffc
fix up outlier packageJsons
sofisl 5f29fcf
update npm link command
sofisl 5464443
update test commands
sofisl eb78ffd
change testing to actions
sofisl a312cca
update presubmit
sofisl b02415a
less tests for lint
sofisl 1a40715
retry
sofisl 5360579
retry lint
sofisl c31d454
retry
sofisl 3b55548
retry
sofisl bbd4f4c
retry
sofisl 99812ca
potentially remove an extra step
sofisl 0ac43c9
remove soon-ton-to-be deprecated packages
sofisl c3d2e62
update snapshot
sofisl 3529c3c
move unit tests to github actions
sofisl c94e1b7
update header check
sofisl 006d556
fix lint issue
sofisl cffd1c2
make change back to system
sofisl 2864bc3
fix lint and reduce system package command
sofisl 0f3c66c
update SRS
sofisl 52675ff
cleanup changes
sofisl 2c02339
Update deletecloudbuild.sh
sofisl 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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,5 @@ | ||
now=$(date -u -d "-15 min" +"%Y-%m-%dT%H:%M:%SZ") | ||
gcloud builds list --format="value(id)" --filter="substitutions.REF_NAME="$REF_NAME" AND status="WORKING" AND create_time<"$now"" | while IFS= read -r line ; do | ||
echo "gcloud builds cancel "$line"" | ||
gcloud builds cancel "$line" | ||
done |
This file was deleted.
Oops, something went wrong.
This file was deleted.
Oops, something went wrong.
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
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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,28 @@ | ||
const fs = require('fs'); | ||
const path = require('path'); | ||
const cp = require('child_process'); | ||
const dirsToSkip = ['gapic-node-templating', 'typeless-sample-bot']; | ||
|
||
async function main() { | ||
const dirs = fs.readdirSync(path.join(__dirname, 'packages')); | ||
for (const dir of dirs) { | ||
if (!dirsToSkip.includes(dir)) { | ||
let rewrite = false; | ||
let packageJson = JSON.parse(fs.readFileSync(path.join('packages', dir, 'package.json'), 'utf-8')); | ||
if (packageJson.scripts.hasOwnProperty("samples-test")) { | ||
packageJson.scripts["samples-test"] = "cd samples/ && npm link ../ && npm i && npm test" | ||
rewrite = true; | ||
} | ||
if (packageJson.scripts.hasOwnProperty("system-test")) { | ||
packageJson.scripts["system-test"] = "c8 mocha build/system-test" | ||
rewrite = true; | ||
} | ||
if (rewrite) { | ||
fs.writeFileSync(path.join('packages', dir, 'package.json'), JSON.stringify(packageJson, null, 2)); | ||
} | ||
} | ||
|
||
} | ||
} | ||
|
||
main(); |
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
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.
what purpose does this script serve?
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 will delete it! Shouldn't be in here. I'll wait once this is approved to delete it so tests won't rerun.