Merge branch 'master' into merge-conflict-resolution-with-master

This commit is contained in:
Nilesh Sarupriya 2023-09-20 11:37:23 +05:30
commit e0346cf8da
12 changed files with 32 additions and 30 deletions

View File

@ -93,7 +93,7 @@ Object.entries(widgetsToTest).forEach(([widgetSelector, testConfig], index) => {
if (index === 0) {
configureApi();
}
entityExplorer.PinUnpinEntityExplorer(false);
// entityExplorer.PinUnpinEntityExplorer(false);
entityExplorer.DragDropWidgetNVerify(widgetSelector, 300, 200);
entityExplorer.DragDropWidgetNVerify(draggableWidgets.BUTTON, 400, 400);
//entityExplorer.SelectEntityByName(draggableWidgets.BUTTONNAME("1"));
@ -110,7 +110,7 @@ Object.entries(widgetsToTest).forEach(([widgetSelector, testConfig], index) => {
PROPERTY_SELECTOR.TextFieldName,
`{{appsmith.store.textPayloadOnSubmit}}`,
);
entityExplorer.PinUnpinEntityExplorer(true);
// entityExplorer.PinUnpinEntityExplorer(true);
});
it("2. StoreValue should have complete input value", () => {

View File

@ -1,7 +1,7 @@
import homePage from "../../../../locators/HomePage";
import * as _ from "../../../../support/Objects/ObjectsCore";
describe("Visual regression tests", () => {
describe.skip("Visual regression tests", () => {
// for any changes in UI, update the screenshot in snapshot folder, to do so:
// 1. Delete the required screenshot which you want to update.
// 2. Run test in headless mode with any browser

View File

@ -1,6 +1,6 @@
import * as _ from "../../../../support/Objects/ObjectsCore";
describe("Visual tests for datasources", () => {
describe.skip("Visual tests for datasources", () => {
// for any changes in UI, update the screenshot in snapshot folder, to do so:
// 1. Delete the required screenshot which you want to update.
// 2. Run test in headless mode with any browser

View File

@ -3,7 +3,7 @@ import { ObjectsRegistry } from "../../../../support/Objects/Registry";
let jsEditor = ObjectsRegistry.JSEditor,
agHelper = ObjectsRegistry.AggregateHelper;
describe("JSEditor Comment - Visual tests", () => {
describe.skip("JSEditor Comment - Visual tests", () => {
it("1. comments code on the editor", () => {
jsEditor.CreateJSObject(
`export default {

View File

@ -8,7 +8,7 @@ import {
dataSources,
} from "../../../../support/Objects/ObjectsCore";
describe("JSEditor Indendation - Visual tests", () => {
describe.skip("JSEditor Indendation - Visual tests", () => {
it("6. TC 1933 - jSEditor prettify verification on cloned application", () => {
const appname = localStorage.getItem("AppName");
jsEditor.CreateJSObject(

View File

@ -1,6 +1,6 @@
import * as _ from "../../../../support/Objects/ObjectsCore";
describe("JS Editor Save and Auto-indent: Visual tests", () => {
describe.skip("JS Editor Save and Auto-indent: Visual tests", () => {
it("1. Auto indents and saves the code when Ctrl/Cmd+s is pressed", () => {
_.jsEditor.CreateJSObject(
`export default {

View File

@ -1,4 +1,4 @@
describe("Visual regression tests", () => {
describe.skip("Visual regression tests", () => {
// for any changes in UI, update the screenshot in snapshot folder, to do so:
// 1. Delete the required screenshot which you want to update
// 2. Run test in headless mode with chrome (to maintain same resolution in CI)

View File

@ -75,13 +75,13 @@ describe("Camera widget - Image test", () => {
agHelper.AssertExistingToggleState("Mirrored", "true");
propPane.EnterJSContext("Mirrored", "{{(55>45)?false:true}}", true, true);
deployMode.DeployApp(locators._widgetInDeployed(draggableWidgets.CAMERA));
agHelper
/* agHelper
.GetElement(locators._widgetInDeployed(draggableWidgets.CAMERA))
.matchImageSnapshot("cameraImageMirroredScreen", {
failureThreshold: 0.15,
failureThresholdType: "percent",
customDiffConfig: { threshold: 0.15 },
});
}); */
deployMode.NavigateBacktoEditor();
entityExplorer.SelectEntityByName("Camera1");
propPane.EnterJSContext("Mirrored", "", false);
@ -114,9 +114,9 @@ describe("Camera widget - Image test", () => {
//Validate camera screen & icons
agHelper.AssertElementVisibility(widgetLocators.cameraImageVideoOnOffBtn);
agHelper.AssertElementVisibility(widgetLocators.cameraImageVideoDropdown);
agHelper
/* agHelper
.GetElement(locators._widgetInDeployed(draggableWidgets.CAMERA))
.matchImageSnapshot("cameraImageScreen");
.matchImageSnapshot("cameraImageScreen"); */
//Capture image
agHelper.GetNClick(widgetLocators.cameraCaptureBtn);
@ -124,18 +124,18 @@ describe("Camera widget - Image test", () => {
agHelper.AssertElementVisibility(widgetLocators.cameraImageDiscardBtn);
//Validate image in preview screen
agHelper
/* agHelper
.GetElement(locators._widgetInDeployed(draggableWidgets.CAMERA))
.matchImageSnapshot("cameraImagePreviewScreen");
.matchImageSnapshot("cameraImagePreviewScreen"); */
//Save image
agHelper.GetNClick(widgetLocators.cameraSaveBtn);
agHelper.AssertElementVisibility(widgetLocators.cameraRefreshBtn);
//Validate image in refresh screen
agHelper
/* agHelper
.GetElement(locators._widgetInDeployed(draggableWidgets.CAMERA))
.matchImageSnapshot("cameraImageSavedScreen");
.matchImageSnapshot("cameraImageSavedScreen"); */
//Refresh image
agHelper.GetNClick(widgetLocators.cameraRefreshBtn);

View File

@ -58,9 +58,9 @@ describe("Camera widget - Video test", () => {
agHelper.AssertExistingToggleState("Mirrored", "true");
propPane.EnterJSContext("Mirrored", "{{(55>45)?false:true}}", true, true);
deployMode.DeployApp(locators._widgetInDeployed(draggableWidgets.CAMERA));
agHelper
/* agHelper
.GetElement(locators._widgetInDeployed(draggableWidgets.CAMERA))
.matchImageSnapshot("cameraVideoMirroredScreen");
.matchImageSnapshot("cameraVideoMirroredScreen"); */
deployMode.NavigateBacktoEditor();
entityExplorer.SelectEntityByName("Camera1");
propPane.EnterJSContext("Mirrored", "", false);
@ -95,9 +95,9 @@ describe("Camera widget - Video test", () => {
agHelper.AssertElementVisibility(widgetLocators.cameraMicrophoneDropdown);
agHelper.AssertElementVisibility(widgetLocators.cameraVideoOnOffBtn);
agHelper.AssertElementVisibility(widgetLocators.cameraVideoDropdown);
agHelper
/* agHelper
.GetElement(locators._widgetInDeployed(draggableWidgets.CAMERA))
.matchImageSnapshot("cameraVideoScreen");
.matchImageSnapshot("cameraVideoScreen"); */
//Start video recording
agHelper.GetNClick(widgetLocators.cameraCaptureBtn);
@ -110,18 +110,18 @@ describe("Camera widget - Video test", () => {
agHelper.AssertElementVisibility(widgetLocators.cameraVideoPlayBtn);
//Validate video in preview screen
agHelper
/* agHelper
.GetElement(locators._widgetInDeployed(draggableWidgets.CAMERA))
.matchImageSnapshot("cameraVideoPreviewScreen");
.matchImageSnapshot("cameraVideoPreviewScreen"); */
//Save video
agHelper.GetNClick(widgetLocators.cameraSaveBtn);
//Validate video in refresh screen
agHelper.AssertElementVisibility(widgetLocators.cameraRefreshBtn);
agHelper
/* agHelper
.GetElement(locators._widgetInDeployed(draggableWidgets.CAMERA))
.matchImageSnapshot("cameraVideoSavedScreen");
.matchImageSnapshot("cameraVideoSavedScreen"); */
//Refresh video
agHelper.GetNClick(widgetLocators.cameraRefreshBtn);

View File

@ -1,8 +1,9 @@
const explorer = require("../../../../../locators/explorerlocators.json");
const widgetsPage = require("../../../../../locators/Widgets.json");
const commonlocators = require("../../../../../locators/commonlocators.json");
const publish = require("../../../../../locators/publishWidgetspage.json");
import explorer from "../../../../../locators/explorerlocators.json";
import publish from "../../../../../locators/publishWidgetspage.json";
import commonlocators from "../../../../../locators/commonlocators.json";
import widgetsPage from "../../../../../locators/Widgets.json";
import * as _ from "../../../../../support/Objects/ObjectsCore";
const widgetName = "codescannerwidget";
const codeScannerVideoOnPublishPage = `${publish.codescannerwidget} ${commonlocators.codeScannerVideo}`;
const codeScannerDisabledSVGIconOnPublishPage = `${publish.codescannerwidget} ${commonlocators.codeScannerDisabledSVGIcon}`;

View File

@ -664,7 +664,7 @@ public class NewActionServiceCEImpl extends BaseService<NewActionRepository, New
@Override
public Flux<NewAction> findAllById(Iterable<String> id) {
return repository.findAllById(id).flatMap(this::sanitizeAction);
return repository.findAllByIdIn(id).flatMap(this::sanitizeAction);
}
@Override
@ -1908,7 +1908,7 @@ public class NewActionServiceCEImpl extends BaseService<NewActionRepository, New
}
return repository
.findAllById(actionIds)
.findAllByIdIn(actionIds)
.map(newAction -> {
// Update collectionId and defaultCollectionIds in actionDTOs
ActionDTO unpublishedAction = newAction.getUnpublishedAction();
@ -1960,6 +1960,7 @@ public class NewActionServiceCEImpl extends BaseService<NewActionRepository, New
* This method is used to publish actions of an application. It does two things:
* 1. it deletes actions which are deleted from the edit mode.
* 2. It updates actions in bulk by setting publishedAction=unpublishedAction
*
* @param applicationId
* @param permission
* @return