Skip to content

Commit 8f16981

Browse files
authored
Merge branch 'main' into issues/300-regex-for-rbac
2 parents 89a75e0 + 73c7217 commit 8f16981

39 files changed

+1046
-462
lines changed

.github/workflows/branch-deploy.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -62,7 +62,7 @@ jobs:
6262
uses: aws-actions/amazon-ecr-login@v2
6363
- name: Build and push
6464
id: docker_build_and_push
65-
uses: docker/build-push-action@v5
65+
uses: docker/build-push-action@v6
6666
with:
6767
builder: ${{ steps.buildx.outputs.name }}
6868
context: api

.github/workflows/build-public-image.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -61,7 +61,7 @@ jobs:
6161
registry-type: 'public'
6262
- name: Build and push
6363
id: docker_build_and_push
64-
uses: docker/build-push-action@v5
64+
uses: docker/build-push-action@v6
6565
with:
6666
builder: ${{ steps.buildx.outputs.name }}
6767
context: api

.github/workflows/cve_checks.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,7 @@ jobs:
5353
${{ runner.os }}-buildx-
5454
5555
- name: Build docker image
56-
uses: docker/build-push-action@v5
56+
uses: docker/build-push-action@v6
5757
with:
5858
builder: ${{ steps.buildx.outputs.name }}
5959
context: api

.github/workflows/docker_build.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -60,7 +60,7 @@ jobs:
6060
6161
- name: Build docker image
6262
id: docker_build
63-
uses: docker/build-push-action@v5
63+
uses: docker/build-push-action@v6
6464
with:
6565
builder: ${{ steps.buildx.outputs.name }}
6666
context: api

.github/workflows/e2e-run.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@ jobs:
2828
ref: ${{ inputs.sha }}
2929

3030
- name: Set up JDK
31-
uses: actions/setup-java@v3
31+
uses: actions/setup-java@v4
3232
with:
3333
java-version: '21'
3434
distribution: 'zulu'
@@ -76,7 +76,7 @@ jobs:
7676
ref: ${{ inputs.sha }}
7777

7878
- name: Set up JDK
79-
uses: actions/setup-java@v3
79+
uses: actions/setup-java@v4
8080
with:
8181
java-version: '21'
8282
distribution: 'zulu'

.github/workflows/separate_env_public_create.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -62,7 +62,7 @@ jobs:
6262
uses: aws-actions/amazon-ecr-login@v2
6363
- name: Build and push
6464
id: docker_build_and_push
65-
uses: docker/build-push-action@v5
65+
uses: docker/build-push-action@v6
6666
with:
6767
builder: ${{ steps.buildx.outputs.name }}
6868
context: api

api/src/main/java/io/kafbat/ui/model/rbac/Role.java

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -14,10 +14,10 @@ public class Role {
1414
List<Permission> permissions;
1515

1616
public void validate() {
17-
checkArgument(!clusters.isEmpty(), "Role clusters cannot be empty");
18-
checkArgument(!subjects.isEmpty(), "Role subjects cannot be empty");
19-
permissions.forEach(Permission::transform);
17+
checkArgument(clusters != null && !clusters.isEmpty(), "Role clusters cannot be empty");
18+
checkArgument(subjects != null && !subjects.isEmpty(), "Role subjects cannot be empty");
2019
permissions.forEach(Permission::validate);
20+
permissions.forEach(Permission::transform);
2121
subjects.forEach(Subject::validate);
2222
}
2323

build.gradle

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -28,16 +28,14 @@ subprojects {
2828

2929
boolean resolveBooleanProperty(String propertyName, boolean defaultValue = false) {
3030
def propertyValueStr = findProperty(propertyName)
31-
return propertyValueStr == null ? defaultValue : propertyValueStr.toBoolean();
31+
return propertyValueStr == null ? defaultValue : propertyValueStr.toBoolean()
3232
}
3333

3434
ext {
3535
release = resolveBooleanProperty("release")
3636
includeFrontend = resolveBooleanProperty("include-frontend", release)
3737
buildDockerImages = resolveBooleanProperty("build-docker-images", release)
3838
runE2e = resolveBooleanProperty("run-e2e")
39-
e2eHeadlessOff = resolveBooleanProperty("e2e-headless-off")
40-
e2eSelenoidOff = resolveBooleanProperty("e2e-selenoid-off")
4139
}
4240

4341
sonar {

e2e-tests/README.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -40,7 +40,7 @@ docker-compose -f documentation/compose/e2e-tests.yaml up -d
4040
2. To run test suite select its name (options: `regression`, `sanity`, `smoke`) and put it instead %s into command below
4141

4242
```
43-
./mvnw -Dsurefire.suiteXmlFiles='src/test/resources/%s.xml' -f 'e2e-tests' test -Pprod
43+
./gradlew :e2e-tests:test -Prun-e2e=true -Psuite_name=%s
4444
```
4545

4646
### Reporting

e2e-tests/build.gradle

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -52,8 +52,6 @@ test {
5252
suiteXmlFiles = [suitePath.toFile()]
5353
}
5454

55-
systemProperty "headless", e2eHeadlessOff
56-
systemProperty "selenoid", e2eSelenoidOff
5755
systemProperty "allure.results.directory",
5856
project.layout.buildDirectory.dir("allure-results").get().getAsFile().getAbsolutePath()
5957
}

e2e-tests/src/main/java/io/kafbat/ui/screens/connectors/ConnectorDetails.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@ public class ConnectorDetails extends BasePage {
2121
@Step
2222
public ConnectorDetails waitUntilScreenReady() {
2323
waitUntilSpinnerDisappear();
24-
dotMenuBtn.shouldBe(Condition.visible);
24+
taskTab.shouldBe(Condition.visible);
2525
return this;
2626
}
2727

@@ -48,7 +48,7 @@ public ConnectorDetails clickSubmitButton() {
4848

4949
@Step
5050
public ConnectorDetails openDotMenu() {
51-
WebUtil.clickByJavaScript(dotMenuBtn);
51+
WebUtil.clickByActions(dotMenuBtn);
5252
return this;
5353
}
5454

e2e-tests/src/main/java/io/kafbat/ui/screens/schemas/SchemaDetails.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -61,7 +61,7 @@ public SchemaDetails openCompareVersionMenu() {
6161

6262
@Step
6363
public SchemaDetails removeSchema() {
64-
WebUtil.clickByJavaScript(dotMenuBtn);
64+
WebUtil.clickByActions(dotMenuBtn);
6565
removeBtn.shouldBe(Condition.enabled).click();
6666
schemaConfirmBtn.shouldBe(Condition.visible).click();
6767
schemaConfirmBtn.shouldBe(Condition.disappear);

e2e-tests/src/main/java/io/kafbat/ui/screens/topics/TopicDetails.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -91,7 +91,7 @@ public String getSettingsGridValueByKey(String key) {
9191

9292
@Step
9393
public TopicDetails openDotMenu() {
94-
WebUtil.clickByJavaScript(dotMenuBtn);
94+
WebUtil.clickByActions(dotMenuBtn);
9595
return this;
9696
}
9797

e2e-tests/src/main/java/io/kafbat/ui/settings/BaseSource.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,11 +7,12 @@
77
public abstract class BaseSource {
88

99
public static final boolean HEADLESS = parseBoolean(getOptionalString(TRUE, System.getProperty("headless")));
10+
public static final boolean SELENOID = parseBoolean(getOptionalString(TRUE, System.getProperty("selenoid")));
1011
public static final String CLUSTER_NAME = "local";
1112
public static final String CONNECT_NAME = "first";
1213
private static final String LOCAL_HOST = "localhost";
1314
public static final String REMOTE_URL = String.format("http://%s:4444/wd/hub", LOCAL_HOST);
1415
public static final String BASE_API_URL = String.format("http://%s:8080", LOCAL_HOST);
15-
public static final String BASE_HOST = "host.docker.internal";
16+
public static final String BASE_HOST = SELENOID ? "host.docker.internal" : LOCAL_HOST;
1617
public static final String BASE_UI_URL = String.format("http://%s:8080", BASE_HOST);
1718
}

e2e-tests/src/main/java/io/kafbat/ui/settings/drivers/WebDriver.java

Lines changed: 11 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -38,17 +38,17 @@ public static void browserSetup() {
3838
.addArguments("--disable-gpu")
3939
.addArguments("--no-sandbox")
4040
.addArguments("--lang=en_US");
41-
42-
Configuration.remote = BaseSource.REMOTE_URL;
43-
Configuration.remoteConnectionTimeout = 180000;
44-
Configuration.remoteReadTimeout = 180000;
45-
Map<String, Object> selenoidOptions = new HashMap<>();
46-
selenoidOptions.put("enableVNC", true);
47-
selenoidOptions.put("enableLog", true);
48-
selenoidOptions.put("enableVideo", false);
49-
selenoidOptions.put("sessionTimeout", "30m");
50-
chromeOptions.setCapability("selenoid:options", selenoidOptions);
51-
41+
if (BaseSource.SELENOID) {
42+
Configuration.remote = BaseSource.REMOTE_URL;
43+
Configuration.remoteConnectionTimeout = 180000;
44+
Configuration.remoteReadTimeout = 180000;
45+
Map<String, Object> selenoidOptions = new HashMap<>();
46+
selenoidOptions.put("enableVNC", true);
47+
selenoidOptions.put("enableLog", true);
48+
selenoidOptions.put("enableVideo", false);
49+
selenoidOptions.put("sessionTimeout", "30m");
50+
chromeOptions.setCapability("selenoid:options", selenoidOptions);
51+
}
5252
Configuration.browserCapabilities = chromeOptions;
5353
}
5454

frontend/package.json

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -15,17 +15,17 @@
1515
"ace-builds": "1.33.0",
1616
"ajv": "8.8.2",
1717
"ajv-draft-04": "^1.0.0",
18-
"ajv-formats": "2.1.1",
18+
"ajv-formats": "3.0.1",
1919
"json-schema-faker": "0.5.6",
20-
"jsonpath-plus": "10.2.0",
20+
"jsonpath-plus": "10.3.0",
2121
"lossless-json": "2.0.11",
2222
"pretty-ms": "7.0.1",
2323
"react": "18.2.0",
2424
"react-ace": "11.0.1",
2525
"react-datepicker": "7.4.0",
2626
"react-dom": "18.2.0",
2727
"react-error-boundary": "4.0.13",
28-
"react-hook-form": "7.51.3",
28+
"react-hook-form": "7.54.2",
2929
"react-hot-toast": "2.4.1",
3030
"react-is": "18.2.0",
3131
"react-multi-select-component": "4.3.4",
@@ -70,7 +70,7 @@
7070
"@types/react-router-dom": "5.3.3",
7171
"@types/styled-components": "5.1.34",
7272
"@types/testing-library__jest-dom": "5.14.9",
73-
"@typescript-eslint/eslint-plugin": "6.21.0",
73+
"@typescript-eslint/eslint-plugin": "7.0.0",
7474
"@typescript-eslint/parser": "6.21.0",
7575
"@vitejs/plugin-react-swc": "3.6.0",
7676
"dotenv": "16.4.5",
@@ -97,7 +97,7 @@
9797
"ts-node": "10.9.2",
9898
"ts-prune": "0.10.3",
9999
"typescript": "5.3.3",
100-
"vite": "5.2.14",
100+
"vite": "6.2.1",
101101
"vite-plugin-checker": "0.6.4",
102102
"vite-plugin-ejs": "1.7.0",
103103
"vite-tsconfig-paths": "4.3.2",

0 commit comments

Comments
 (0)