This is an automated email from the ASF dual-hosted git repository.
pabloem pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/beam.git
The following commit(s) were added to refs/heads/master by this push:
new 45955da Merge pull request #16117 from
[BEAM-13368][Playground][Bugfix] Fix CI and failed unit tests on master
45955da is described below
commit 45955daa14be8fec7e986474c2fef1622c13664c
Author: Artur Khanin <[email protected]>
AuthorDate: Fri Dec 3 18:42:11 2021 +0300
Merge pull request #16117 from [BEAM-13368][Playground][Bugfix] Fix CI and
failed unit tests on master
* Fixed RunCode test in controller_test.go
* Update tests for backend
Co-authored-by: Sergey Kalinin <[email protected]>
---
playground/backend/build.gradle.kts | 2 +-
playground/backend/cmd/server/controller_test.go | 21 +++++++--------------
2 files changed, 8 insertions(+), 15 deletions(-)
diff --git a/playground/backend/build.gradle.kts
b/playground/backend/build.gradle.kts
index e73b00e..be66356 100644
--- a/playground/backend/build.gradle.kts
+++ b/playground/backend/build.gradle.kts
@@ -46,7 +46,7 @@ task("test") {
doLast {
exec {
executable("go")
- args("test", "internal/...")
+ args("test", "./...")
}
}
}
diff --git a/playground/backend/cmd/server/controller_test.go
b/playground/backend/cmd/server/controller_test.go
index c7ddbd6..8e7b730 100644
--- a/playground/backend/cmd/server/controller_test.go
+++ b/playground/backend/cmd/server/controller_test.go
@@ -125,10 +125,9 @@ func TestPlaygroundController_RunCode(t *testing.T) {
request *pb.RunCodeRequest
}
tests := []struct {
- name string
- args args
- wantStatus pb.Status
- wantErr bool
+ name string
+ args args
+ wantErr bool
}{
{
// Test case with calling RunCode method with incorrect
SDK.
@@ -154,8 +153,7 @@ func TestPlaygroundController_RunCode(t *testing.T) {
Sdk: pb.Sdk_SDK_JAVA,
},
},
- wantStatus: pb.Status_STATUS_COMPILING,
- wantErr: false,
+ wantErr: false,
},
}
for _, tt := range tests {
@@ -175,19 +173,14 @@ func TestPlaygroundController_RunCode(t *testing.T) {
t.Errorf("PlaygroundController_RunCode() response shoudn't be nil")
} else {
if response.PipelineUuid == "" {
-
t.Errorf("PlaygroundController_RunCode() response.pipeLineId shoudn't be nil")
- } else {
- path :=
os.Getenv("APP_WORK_DIR") + "/executable_files"
- os.RemoveAll(path)
+
t.Errorf("PlaygroundController_RunCode() response.pipelineId shoudn't be nil")
}
status, _ :=
cacheService.GetValue(tt.args.ctx, uuid.MustParse(response.PipelineUuid),
cache.Status)
+ path := os.Getenv("APP_WORK_DIR") +
"/executable_files"
+ os.RemoveAll(path)
if status == nil {
t.Errorf("PlaygroundController_RunCode() status shoudn't be nil")
}
- if !reflect.DeepEqual(status,
tt.wantStatus) {
-
t.Errorf("PlaygroundController_RunCode() status = %v, wantStatus %v", status,
tt.wantStatus)
- }
-
}
}
})
