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

Implement per-service test setup #1985

Merged
merged 102 commits into from
Jan 15, 2025
Merged
Changes from 1 commit
Commits
Show all changes
102 commits
Select commit Hold shift + click to select a range
e581529
Implement per-service test setup
LarryOsterman Jan 9, 2025
d9cdb8e
Second try for package location
LarryOsterman Jan 9, 2025
b34e7c2
Third try for package location
LarryOsterman Jan 9, 2025
7865d00
Fourth try for package location
LarryOsterman Jan 9, 2025
3f73951
Fifth try for package location
LarryOsterman Jan 9, 2025
dbfbb07
Sixth try for package location
LarryOsterman Jan 9, 2025
67f8e8b
Seventh try for package location
LarryOsterman Jan 9, 2025
ea900ba
Next try for package location
LarryOsterman Jan 9, 2025
af0a237
Next try for package location
LarryOsterman Jan 9, 2025
d6c9e31
Next try for package location
LarryOsterman Jan 9, 2025
a55d07e
Next try for package location
LarryOsterman Jan 9, 2025
e18b4ac
Next try for package location
LarryOsterman Jan 9, 2025
a0ed88d
Next try for package location
LarryOsterman Jan 9, 2025
11cf159
Next try for package location
LarryOsterman Jan 9, 2025
5661c76
Next try for package location
LarryOsterman Jan 9, 2025
3982ad6
Next try for package location
LarryOsterman Jan 9, 2025
58f681d
Next try for package location
LarryOsterman Jan 9, 2025
217e9aa
Next try for package location
LarryOsterman Jan 9, 2025
96e6287
Next try for package location
LarryOsterman Jan 9, 2025
4165061
Next try for package location
LarryOsterman Jan 9, 2025
b5ce96f
Next try for package location
LarryOsterman Jan 9, 2025
153c267
Need to check for OSX as well as Linux on chmod
LarryOsterman Jan 9, 2025
bc18572
For reasons the URL parsing doesn't work unless you're in the azure-a…
LarryOsterman Jan 9, 2025
ac3c1d1
Try heath's suggestion w.r.t. the URL parameter
LarryOsterman Jan 9, 2025
9de7977
Fix execution of cleanup script
LarryOsterman Jan 9, 2025
c601de7
Print out TestAmqpBroker process names
LarryOsterman Jan 9, 2025
4e30a29
Try running test broker from broker directory directly
LarryOsterman Jan 9, 2025
bbf828e
Split out windows and linux launching
LarryOsterman Jan 10, 2025
c2acb39
Split out windows and linux launching
LarryOsterman Jan 10, 2025
6ebe406
Split out windows and linux launching 3
LarryOsterman Jan 10, 2025
35d2a9a
Split out windows and linux launching 4
LarryOsterman Jan 10, 2025
3ae0fb4
Removed reference to broker Executable
LarryOsterman Jan 10, 2025
308e0b3
Try a new dotnet run invocation
LarryOsterman Jan 10, 2025
1a01b94
Try a different .net invocation
LarryOsterman Jan 10, 2025
7f5409c
Diagnostics for stopping the test job.
LarryOsterman Jan 10, 2025
496b5bb
Align windows and other platforms
LarryOsterman Jan 10, 2025
c818c25
Different test command line.
LarryOsterman Jan 10, 2025
e11dfc5
Try not using the -- separator
LarryOsterman Jan 10, 2025
aeb5a48
Try not using the -- separator 2
LarryOsterman Jan 10, 2025
61217fa
Add dotnet publish step
LarryOsterman Jan 10, 2025
38f7d4e
Fixed typo in dotnet exec command line; Dump attributes for test broker
LarryOsterman Jan 10, 2025
ef2a6e5
Moved test scripts to per-package directory
LarryOsterman Jan 10, 2025
c825315
Fixed missing trailing quote
LarryOsterman Jan 10, 2025
3901901
Fixed script path after changes
LarryOsterman Jan 10, 2025
963925e
Simpler path commit
LarryOsterman Jan 10, 2025
26e3a27
Fixed relative path to eng directory
LarryOsterman Jan 10, 2025
e46c1ee
Use Path.Combine instead of concatenation
LarryOsterman Jan 10, 2025
09e6b95
Use Path.Combine correctly
LarryOsterman Jan 10, 2025
04f6c1e
Create working directory if it does not exist
LarryOsterman Jan 10, 2025
461b8d8
Create directory correctly
LarryOsterman Jan 10, 2025
84721d2
Stop using dotnet run to launch broker
LarryOsterman Jan 10, 2025
e2d4b0f
Added diagnostics on stopping the test script
LarryOsterman Jan 10, 2025
0da5871
Test configuration is per-package so don't need to check the package
LarryOsterman Jan 10, 2025
d3c64c5
Don't check working directory parameter and stop checking package nam…
LarryOsterman Jan 10, 2025
54a3e0e
Refined checks for job state
LarryOsterman Jan 10, 2025
8df5430
Diagnostics
LarryOsterman Jan 10, 2025
9414a88
More start job diagnostics
LarryOsterman Jan 10, 2025
b744fb1
Another command line broker fix
LarryOsterman Jan 10, 2025
e118920
Another try at finding job output
LarryOsterman Jan 10, 2025
af6d7e3
Try adding a path to the start of the test amqp broker command line
LarryOsterman Jan 10, 2025
18e668e
Sleep after job startup to see state at launch
LarryOsterman Jan 10, 2025
b431483
Try self-contained builds
LarryOsterman Jan 10, 2025
5ee2066
Use selfcontained package directory
LarryOsterman Jan 10, 2025
d5b8694
Forgot to delete a line
LarryOsterman Jan 10, 2025
ca113e0
Removed a couple of variables that are no longer needed
LarryOsterman Jan 10, 2025
3f0f936
Fixed variable name
LarryOsterman Jan 10, 2025
3c7d1b2
AMQP Connection tests use test broker
LarryOsterman Jan 10, 2025
7689185
Ooops, accidentally broke the connect with error test :).
LarryOsterman Jan 10, 2025
9f5476b
Tests now run (fixed silly typo)
LarryOsterman Jan 10, 2025
3c9e2a6
PR feedback
LarryOsterman Jan 13, 2025
07c2c19
Undid some PR feedback; pin to release 2.6.9 of the azure-amqp reposi…
LarryOsterman Jan 13, 2025
0190ce9
Added test instructions; Fixed git clone command
LarryOsterman Jan 13, 2025
dbe0a0f
Added back manual dotnet restore
LarryOsterman Jan 13, 2025
f6239e4
azure-amqp version 2.6.9 (most recent release) apparently references …
LarryOsterman Jan 13, 2025
c8ddc34
Fixed git clone variable
LarryOsterman Jan 13, 2025
e467d3e
Removed individual build steps and just use dotnet publish
LarryOsterman Jan 13, 2025
a43a4cb
Reduced references to Job IDs in script output
LarryOsterman Jan 13, 2025
c896357
Try to run TestAmqpBroker without dotnet publish
LarryOsterman Jan 13, 2025
e980b96
TestAmqpBroker try 2
LarryOsterman Jan 13, 2025
b218756
Try TestAmqpBroker with no extension and a runtimeconfig file
LarryOsterman Jan 13, 2025
720fc68
Not sure what's up right now
LarryOsterman Jan 13, 2025
5534819
Removed --runtimeconfig switch
LarryOsterman Jan 13, 2025
9c70bc6
Try .Dll extension
LarryOsterman Jan 13, 2025
4c2eeaf
Try .Dll extension
LarryOsterman Jan 13, 2025
ec78db9
another attempt at dotnet exec command line.
LarryOsterman Jan 14, 2025
bed6ff4
Try rolling forward to the latest dotnet
LarryOsterman Jan 14, 2025
1c29e22
Fixed roll-forward switch name
LarryOsterman Jan 14, 2025
51fada4
Applyh --roll-forward to dotnet exec not dotnet build
LarryOsterman Jan 14, 2025
2d433fa
Roll forward as a property
LarryOsterman Jan 14, 2025
59ab7c4
Simplify expression to confirm test broker is still running
LarryOsterman Jan 14, 2025
fc3b503
Removed test package parameter from test scripts
LarryOsterman Jan 15, 2025
0586642
Removed package name parameter to test script invocation
LarryOsterman Jan 15, 2025
ee939cc
PR feedback
LarryOsterman Jan 15, 2025
6e024a3
Use hotfix branch, not main
LarryOsterman Jan 15, 2025
a0d5efc
Use master branch after all
LarryOsterman Jan 15, 2025
e25e523
Run example tests in parallel
LarryOsterman Jan 15, 2025
b65b594
Changed output level of connection ignored error
LarryOsterman Jan 15, 2025
6894273
Don't log test cleanup script name
LarryOsterman Jan 15, 2025
2fc0ea0
Fixed slash direction in powershell snippet
LarryOsterman Jan 15, 2025
12be0e7
Fixed dotnet name.
LarryOsterman Jan 15, 2025
12cfb85
Cleaned up language around running the test scripts
LarryOsterman Jan 15, 2025
4cd4702
Removed one spurious .net reference
LarryOsterman Jan 15, 2025
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion eng/scripts/Test-Packages.ps1
Original file line number Diff line number Diff line change
Expand Up @@ -52,7 +52,7 @@ foreach ($package in $packagesToTest) {
Write-Host "Checking for setup in $serviceDirectory + 'Test-Setup.ps1'"
if (Test-Path ($serviceDirectory + "Test-Setup.ps1")) {
Write-Host "`n`nRunning test setup script for package: '$($package.Name)': ($serviceDirectory)Test-Setup.ps1`n"
Invoke-LoggedCommand "($serviceDirectory)Test-Setup.ps1 -packageName $package.Name -workingDirectory $WorkingDirectory"
Invoke-LoggedCommand "($serviceDirectory)Test-Setup.ps1 -packageName ($package.Name) -workingDirectory $WorkingDirectory"
LarryOsterman marked this conversation as resolved.
Show resolved Hide resolved
if ($LASTEXITCODE -ne 0) {
LarryOsterman marked this conversation as resolved.
Show resolved Hide resolved
Write-Error "Test setup script failed for package: '$($package.Name)'"
exit 1
Expand Down
Loading