-
Notifications
You must be signed in to change notification settings - Fork 49
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
Rerun suite, test, url #412
Changes from all commits
a381d76
386273c
1526443
2e096a5
010593b
a6198eb
556f061
e0bd59b
fbd14d9
956dcfa
278d22b
c22ccee
f9d9660
aa1d99d
fd39331
20395cb
48c1563
b421341
73b0abd
6715f59
988f255
c020d2f
d4b3350
f7ca219
bdf2fb9
0765aae
deb4cc9
140869e
0e9c74a
fb8e612
a00b8a1
130a76b
a8de89b
ba2cd9d
53d9d97
ea87d9d
ee4c2ad
b5e85cb
f8d728d
89410ae
903b24b
8f07000
cd9f968
130a069
94ded09
03b8ea3
2d03479
05dbffd
3e0303d
0af1cef
c5d11b1
586cf82
87f6567
3115a95
4bfedf0
1e367f6
af28aa4
47acf66
c1e494b
b72b6ae
dfd2e0d
e22bb69
e5f88e5
ce37513
91f2305
71c089a
82f76e0
bdc4bd0
f88ab76
2b1c330
521428a
d7d2fd1
efd6fb2
374153f
831ec35
bdee2da
f443d1c
c500017
da0a5b9
ce07e0b
ad8b522
02876c2
5e56e20
cb735c3
837feb0
a59978f
09d6672
58fd3a3
50e368d
b2a0afd
93e9436
1d6be4e
fa10a6e
5450a7c
72b58a8
4c1bcd7
9a4b71f
6ec9096
79ec038
a6701df
1146fa3
72aa7fa
4972647
edd6c5d
7be20ba
231ea33
520402d
23bda3a
9b338cd
54e7c3f
883f833
c5f892c
a98d951
cb45bb3
e954ac7
4dc414d
d089a07
acf4437
6335fb5
3112cd4
abcaed8
90ffdf5
c9c141d
aaf3ab3
8d9fcd5
9f4c22a
4909046
bc500dc
8c1d52a
648f823
0fee1db
624da29
aeeefc5
4b66674
39bb767
99959f1
b72a58b
39c1554
fcbb7c2
be44014
188ed1a
eac01e3
4abab31
ab35260
b47ba4b
d915dd3
c98ea26
02c8c98
4a9fbf6
caf218e
36420f8
49383c0
9688e00
91ed8c4
98c81e2
f93d800
9028638
e44f548
b07fa67
dc50bd5
6c648e5
4d27c1b
6b81bba
08bce92
6b8667a
c8451cf
278227b
05a60cd
f20cf28
0c2cd28
9bc6cde
9af85c2
5f60048
74dba89
3e10020
c97b06c
3a2f6ec
5e1c2c3
dbc5a0d
4403542
ad910a5
69e6edd
5621f56
7eefd43
cfb375f
9b63cd6
fa6784d
bdbf8eb
37c8079
87fede7
dedc37c
1878579
3143b41
d33ea41
7fb9c9b
b7d12b5
3a32391
7ae5857
7a2bbf2
537c3c6
3fd1cb1
c2ad82f
17d9c71
b3a3eff
27b5f3f
df2edf4
741e59f
bc95428
67955be
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,46 @@ | ||
/** | ||
* AET | ||
* | ||
* Copyright (C) 2013 Cognifide Limited | ||
* | ||
* Licensed under the Apache License, Version 2.0 (the "License"); you may not use this file except | ||
* in compliance with the License. You may obtain a copy of the License at | ||
* | ||
* http://www.apache.org/licenses/LICENSE-2.0 | ||
* | ||
* Unless required by applicable law or agreed to in writing, software distributed under the License | ||
* is distributed on an "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express | ||
* or implied. See the License for the specific language governing permissions and limitations under | ||
* the License. | ||
*/ | ||
package com.cognifide.aet.communication.api.messages; | ||
|
||
import java.io.Serializable; | ||
import java.util.Arrays; | ||
import org.apache.commons.lang3.StringUtils; | ||
|
||
public class FullProgressLog implements Serializable { | ||
|
||
private static final long serialVersionUID = -7331561874304014158L; | ||
|
||
private ProgressLog compareLog; | ||
private ProgressLog collectLog; | ||
|
||
public FullProgressLog(ProgressLog collectLog, ProgressLog compareLog) { | ||
this.collectLog = collectLog; | ||
this.compareLog = compareLog; | ||
} | ||
|
||
public ProgressLog getCompareLog() { | ||
return compareLog; | ||
} | ||
|
||
public ProgressLog getCollectLog() { | ||
return collectLog; | ||
} | ||
|
||
@Override | ||
public String toString() { | ||
return StringUtils.join(Arrays.asList(compareLog, collectLog), " ::: "); | ||
} | ||
} |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,22 @@ | ||
/** | ||
* AET | ||
* | ||
* Copyright (C) 2013 Cognifide Limited | ||
* | ||
* Licensed under the Apache License, Version 2.0 (the "License"); you may not use this file except | ||
* in compliance with the License. You may obtain a copy of the License at | ||
* | ||
* http://www.apache.org/licenses/LICENSE-2.0 | ||
* | ||
* Unless required by applicable law or agreed to in writing, software distributed under the License | ||
* is distributed on an "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express | ||
* or implied. See the License for the specific language governing permissions and limitations under | ||
* the License. | ||
*/ | ||
package com.cognifide.aet.communication.api.metadata; | ||
|
||
public enum RunType { | ||
SUITE, | ||
TEST, | ||
URL, | ||
} |
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -32,6 +32,7 @@ | |
import java.util.Collection; | ||
import java.util.List; | ||
import java.util.Map; | ||
import java.util.Optional; | ||
import java.util.Set; | ||
import javax.validation.ConstraintViolation; | ||
import javax.validation.Valid; | ||
|
@@ -43,7 +44,8 @@ | |
|
||
public class Suite implements Serializable, Commentable, Named, Validatable { | ||
|
||
private static final long serialVersionUID = 3602287822306302730L; | ||
private static final long serialVersionUID = -3225670696134184553L; | ||
|
||
private static final Gson GSON_FOR_JSON = new GsonBuilder() | ||
.registerTypeHierarchyAdapter(Collection.class, new CollectionSerializer()) | ||
.registerTypeHierarchyAdapter(Map.class, new MapSerializer()) | ||
|
@@ -53,7 +55,7 @@ public class Suite implements Serializable, Commentable, Named, Validatable { | |
}.getType(); | ||
|
||
@NotBlank | ||
private final String correlationId; | ||
private String correlationId; | ||
|
||
@NotBlank | ||
@Size(max = 30) | ||
|
@@ -130,6 +132,17 @@ public List<Test> getTests() { | |
return tests; | ||
} | ||
|
||
public Optional<Test> getTest(String testName){ | ||
Test testToReturn = null; | ||
for (Test test: this.tests) { | ||
if(test.getName().equals(testName)){ | ||
testToReturn = test; | ||
break; | ||
} | ||
} | ||
return Optional.ofNullable(testToReturn); | ||
} | ||
|
||
public boolean addTest(Test test) { | ||
return tests.add(test); | ||
} | ||
|
@@ -191,10 +204,14 @@ public String toString() { | |
.toString(); | ||
} | ||
|
||
public void setVersion(long version) { | ||
public void setVersion(Long version) { | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. is it expected that There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Yes, |
||
this.version = version; | ||
} | ||
|
||
public void setCorrelationId(String correlationId) { | ||
this.correlationId = correlationId; | ||
} | ||
|
||
@Override | ||
public void validate(final Set<String> ignoreFields) throws ValidatorException { | ||
Set<ConstraintViolation<Suite>> errors = FluentIterable | ||
|
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.
is it possible to pass the condition?
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.
Thank you for the review ;)
Unfortunately, it is possible because the model in mongo has null when you saving empty array.
You can find more details here: discussion