Skip to content

Commit cf51a7e

Browse files
authored
Update codeowners (#809)
1 parent ef50cab commit cf51a7e

File tree

8 files changed

+48
-19
lines changed

8 files changed

+48
-19
lines changed

.github/CODEOWNERS

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,4 +11,4 @@
1111
# In each subsection folders are ordered first by depth, then alphabetically.
1212
# This should make it easy to add new rules without breaking existing ones.
1313

14-
* @grafana/aws-datasources
14+
* @grafana/data-sources-plugins

.github/workflows/publish.yaml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -40,7 +40,6 @@ jobs:
4040
ACCESS_KEY=e2e:accessKey
4141
SECRET_KEY=e2e:secretKey
4242
github-draft-release: false # publish the github release directly, skipping the draft step
43-
run-playwright-with-skip-grafana-react-19-preview-image: true
4443

4544
# Scope for the plugin published to the catalog. Setting this to "grafana_cloud" will make it visible only in Grafana Cloud
4645
# (and hide it for on-prem). This is required for some provisioned plugins.

.github/workflows/push.yaml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,4 +21,3 @@ jobs:
2121
playwright-secrets: |
2222
ACCESS_KEY=e2e:accessKey
2323
SECRET_KEY=e2e:secretKey
24-
run-playwright-with-skip-grafana-react-19-preview-image: true

package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,7 @@
4141
"@eslint/js": "9.39.2",
4242
"@grafana/aws-sdk": "0.10.2",
4343
"@grafana/eslint-config": "9.0.0",
44-
"@grafana/plugin-e2e": "3.4.5",
44+
"@grafana/plugin-e2e": "3.4.12",
4545
"@grafana/tsconfig": "2.0.1",
4646
"@openfeature/core": "1.9.2",
4747
"@openfeature/web-sdk": "1.7.3",

playwright.config.ts

Lines changed: 11 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -36,9 +36,19 @@ export default defineConfig<PluginOptions>({
3636
use: {
3737
/* Base URL to use in actions like `await page.goto('/')`. */
3838
baseURL: 'http://localhost:3000',
39+
featureToggles: {
40+
dashboardNewLayouts: false,
41+
multiPropsVariables: false,
42+
},
43+
openFeature: {
44+
flags: {
45+
splashScreen: false,
46+
},
47+
},
3948

4049
/* Collect trace when retrying the failed test. See https://playwright.dev/docs/trace-viewer */
41-
trace: 'on-first-retry',
50+
trace: 'on',
51+
video: 'on',
4252
},
4353

4454
/* Configure projects for major browsers */

src/ConfigEditor/ConfigEditor.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,7 @@ type Workgroup = {
4141
type InputResourceType = 'dbUser' | 'database';
4242

4343
export function ConfigEditor(props: Props) {
44-
const baseURL = `/api/datasources/${props.options.id}`;
44+
const baseURL = `/api/datasources/uid/${props.options.uid}`;
4545
const resourcesURL = `${baseURL}/resources`;
4646
const { jsonData } = props.options;
4747
const [saved, setSaved] = useState(!!jsonData.defaultRegion);

tests/alertRule.spec.ts

Lines changed: 24 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -14,9 +14,30 @@ test('should successfully create an alert rule', async ({
1414
}) => {
1515
const ds = await readProvisionedDataSource({ fileName: 'aws-redshift-e2e.yaml', name: 'AWS Redshift E2E' });
1616
const queryA = alertRuleEditPage.getAlertRuleQueryRow('A');
17-
await queryA.datasource.set(ds.name);
18-
await page.waitForFunction(() => window.monaco);
19-
await queryA.getByGrafanaSelector(selectors.components.CodeEditor.container).click();
17+
18+
try {
19+
await queryA.datasource.set(ds.name);
20+
await page.waitForFunction(() => window.monaco);
21+
await queryA.getByGrafanaSelector(selectors.components.CodeEditor.container).click();
22+
} catch {
23+
// TODO: Remove this fallback once @grafana/plugin-e2e picks up the Grafana 13 query-row selectors.
24+
const queryEditorRow = alertRuleEditPage.getByGrafanaSelector('data-testid Query editor row').filter({
25+
has: alertRuleEditPage.getByGrafanaSelector('data-testid Query editor row title A'),
26+
});
27+
const datasourcePicker = queryEditorRow.getByTestId(selectors.components.DataSourcePicker.inputV2);
28+
29+
await expect(datasourcePicker).toBeVisible();
30+
await datasourcePicker.fill(ds.name);
31+
await page.keyboard.press('ArrowDown');
32+
await page.keyboard.press('ArrowUp');
33+
await page.keyboard.press('Enter');
34+
35+
await page.waitForFunction(() => window.monaco);
36+
await alertRuleEditPage
37+
.getByGrafanaSelector(selectors.components.CodeEditor.container, { root: queryEditorRow })
38+
.click();
39+
}
40+
2041
await page.keyboard.insertText('SELECT environment, temperature FROM public.long_format_example limit 2');
2142
await expect(alertRuleEditPage.evaluate()).toBeOK();
2243
});

yarn.lock

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -1531,14 +1531,14 @@ __metadata:
15311531
languageName: node
15321532
linkType: hard
15331533

1534-
"@grafana/e2e-selectors@npm:13.0.0-22898798261":
1535-
version: 13.0.0-22898798261
1536-
resolution: "@grafana/e2e-selectors@npm:13.0.0-22898798261"
1534+
"@grafana/e2e-selectors@npm:13.0.0-23624974663":
1535+
version: 13.0.0-23624974663
1536+
resolution: "@grafana/e2e-selectors@npm:13.0.0-23624974663"
15371537
dependencies:
15381538
semver: "npm:^7.7.0"
15391539
tslib: "npm:2.8.1"
15401540
typescript: "npm:5.9.2"
1541-
checksum: 10c0/d437e29cb25e4974088ab9108d0489dd3c9fb599d03b46a480814bcdb36ee62bae6b5cad865137665ae8d1d0a23eb74b9e8cee34f386a5738e68eaddeb53bd36
1541+
checksum: 10c0/714095fbf1932c49a923027e0d6d5e6d89f7dd06070e512716f58a7a2241eeac10b24ef9427b5ff5a21b2e4df43240ed2532676197e5daa75e7828b044e7bc54
15421542
languageName: node
15431543
linkType: hard
15441544

@@ -1598,11 +1598,11 @@ __metadata:
15981598
languageName: node
15991599
linkType: hard
16001600

1601-
"@grafana/plugin-e2e@npm:3.4.5":
1602-
version: 3.4.5
1603-
resolution: "@grafana/plugin-e2e@npm:3.4.5"
1601+
"@grafana/plugin-e2e@npm:3.4.12":
1602+
version: 3.4.12
1603+
resolution: "@grafana/plugin-e2e@npm:3.4.12"
16041604
dependencies:
1605-
"@grafana/e2e-selectors": "npm:13.0.0-22898798261"
1605+
"@grafana/e2e-selectors": "npm:13.0.0-23624974663"
16061606
semver: "npm:^7.5.4"
16071607
uuid: "npm:^13.0.0"
16081608
yaml: "npm:^2.3.4"
@@ -1612,7 +1612,7 @@ __metadata:
16121612
peerDependenciesMeta:
16131613
"@axe-core/playwright":
16141614
optional: true
1615-
checksum: 10c0/f125153cd7ff222e24c418bff805263872d2c0012a24ecf83f00f7b9e0a3369b2283e04a3d234d03c43aec779ce7e6aed9de418ebb9769233589937e02c8760a
1615+
checksum: 10c0/b87340d2cb04bcbcaf7676d61f8f990fd26ba1d8f53cfb922e89a4751ede29bbe61021d770d3a9ebdf4500fe404fe6512ddea03be4d1c08ee15d22e9e5a376e7
16161616
languageName: node
16171617
linkType: hard
16181618

@@ -7508,7 +7508,7 @@ __metadata:
75087508
"@grafana/aws-sdk": "npm:0.10.2"
75097509
"@grafana/data": "npm:12.4.1"
75107510
"@grafana/eslint-config": "npm:9.0.0"
7511-
"@grafana/plugin-e2e": "npm:3.4.5"
7511+
"@grafana/plugin-e2e": "npm:3.4.12"
75127512
"@grafana/plugin-ui": "npm:^0.13.0"
75137513
"@grafana/runtime": "npm:12.4.1"
75147514
"@grafana/schema": "npm:12.4.1"

0 commit comments

Comments
 (0)