Commit Graph

50 Commits

Author SHA1 Message Date
Saroj
e1c8012074
ci: Fix for dependency installation time (#23239)
## Description
- Added --immutable to yarn install

#### Type of change
- Workflow files

#### How Has This Been Tested?
- Manual
2023-05-11 22:26:52 +05:30
Satish Gandham
83538ad74d
feat: Bundle optimization and first load improvements (#21667)
Co-authored-by: Ivan Akulov <mail@iamakulov.com>
Co-authored-by: Satish Gandham <hello@satishgandham.com>
Co-authored-by: Ivan Akulov <iamakulov@outlook.com>
Co-authored-by: Aishwarya UR <aishwarya@appsmith.com>
Co-authored-by: Shrikant Sharat Kandula <shrikant@appsmith.com>
Co-authored-by: somangshu <somangshu.goswami1508@gmail.com>
2023-05-11 10:56:03 +05:30
Shrikant Sharat Kandula
1836a44577
ci: Remove license in ci-test.yml (#21938)
The License key is unused in this repo. Setting it here is misleading,
and can cause confusions at best, or missed accidental License failures
at worst.

This PR removes the License key occurrences from `ci-test.yml`.

Note: Please review/approve only. Don't merge. This will likely cause a
conflict, so I'd like to be available when this is merged so I can
resolve the conflict. Thanks!
2023-04-05 09:38:44 +05:30
Aishwarya-U-R
85f3ebc1f3
ci: Starting ElasticSearch DB container for running test scripts in CI runs (#22102)
## Description

- This PR includes ElasticSearch as separate docker container inside CI
that can be used for running ElasticSearch datasource test cases in CI

## Type of change

- Yaml file update


## Checklist:
### QA activity:
- [ ] Added Test Plan Approved label after reviewing all changes
2023-04-05 01:36:57 +05:30
Aishwarya UR
a5f0689a24 Adding FIRESTORE_PRIVATE_KEY to git 2023-04-04 03:12:46 +05:30
Saroj
a5aabe7bbc
ci: Upload an empty failed-spec-ci file if all the specs are passed (#21914)
## Description
-  Upload an empty failed-spec-ci file if all the specs are  passed

## Type of change
- YML file changes

## How Has This Been Tested?
- github actions

## Checklist:
### QA activity:
- [ ] Test plan has been approved by relevant developers
- [ ] Test plan has been peer reviewed by QA
- [ ] Cypress test cases have been added and approved by either SDET or
manual QA
- [ ] Organized project review call with relevant stakeholders after
Round 1/2 of QA
- [ ] Added Test Plan Approved label after reveiwing all Cypress test
2023-03-30 20:14:22 +05:30
Saroj
ee3bccdae2
ci: Resetting the failed_spec_ci in case of success in ci-test.yml (#21853)
## Description
- Resetting the failed_spec_ci in case of success in ci-test.yml

## Type of change
- yml file changes

## How Has This Been Tested?
- Workflow run

## Checklist:
### QA activity:
- [ ] Test plan has been approved by relevant developers
- [ ] Test plan has been peer reviewed by QA
- [ ] Cypress test cases have been added and approved by either SDET or
manual QA
- [ ] Organized project review call with relevant stakeholders after
Round 1/2 of QA
- [ ] Added Test Plan Approved label after reveiwing all Cypress test
2023-03-29 13:30:30 +05:30
Aishwarya-U-R
eaede7960e
ci: Starting Arango DB container for running Arango test scripts in CI runs (#21841)
## Description

- This PR includes Arango as separate docker container inside CI that
can be used for running Arango datasource test cases in CI

## Type of change

- Yml file update


## Checklist:
- [X] Added Test Plan Approved label after reviewing all changes
2023-03-29 00:01:53 +05:30
Aishwarya-U-R
413d430025
test: Cypress - Segment calls intercepted (#21726)
## Description

- This PR adds the host for segment calls as localhost as opposed to
sending it to Segment itself
- Diverting segment calls to Localhost for Cypress CI runs &
intercepting the response
- Updated to Analytics spec to validate the enableTelemtry false state
- Bug 21191, validation included
- Welcome screen - not toggling off the Usage Details anymore

## Type of change

- Yml file update
- Script updates


## Checklist:
### QA activity:
- [X] Added Test Plan Approved label after reviewing all changes
2023-03-24 21:36:48 +05:30
Saroj
608658818f
ci: Added condition to fetch the specs failed in after hook in ci-test.yml (#21713)
## Description
- Added condition to fetch the specs failed in after hook in ci-test.yml

## Type of change
- YML file changes
- 
## How Has This Been Tested?
- Manual

## Checklist:
### QA activity:
- [ ] Test plan has been approved by relevant developers
- [ ] Test plan has been peer reviewed by QA
- [ ] Cypress test cases have been added and approved by either SDET or
manual QA
- [ ] Organized project review call with relevant stakeholders after
Round 1/2 of QA
- [ ] Added Test Plan Approved label after reveiwing all Cypress test
2023-03-23 13:11:02 +05:30
Saroj
f17df15953
ci: Fix for combined-failed-spec-ci cache not found issue (#21706)
## Description
- Fix for combined-failed-spec-ci cache not found issue

## Type of change
- YML file changes

## How Has This Been Tested?
- Manual

## Checklist:
### QA activity:
- [ ] Test plan has been approved by relevant developers
- [ ] Test plan has been peer reviewed by QA
- [ ] Cypress test cases have been added and approved by either SDET or
manual QA
- [ ] Organized project review call with relevant stakeholders after
Round 1/2 of QA
- [ ] Added Test Plan Approved label after reveiwing all Cypress test
2023-03-23 10:51:41 +05:30
Saroj
bc79466a99
ci: Updated the workflow files to use actions/cache@v3 (#21669)
## Description
- Updated the workflow files to use actions/cache@v3 

## Type of change
- yml file changes

## How Has This Been Tested?
- Manual

## Checklist:
### QA activity:
- [ ] Test plan has been approved by relevant developers
- [ ] Test plan has been peer reviewed by QA
- [ ] Cypress test cases have been added and approved by either SDET or
manual QA
- [ ] Organized project review call with relevant stakeholders after
Round 1/2 of QA
- [ ] Added Test Plan Approved label after reveiwing all Cypress test
2023-03-22 16:17:13 +05:30
Saroj
a872435320
ci: Reverting the use of combined-failed-spec-ci artifact instead of cache (#21663)
## Description
- Reverting the use of combined-failed-spec-ci artifact instead of cache

## Type of change
- yml file changes

## How Has This Been Tested?
- Manual

## Checklist:
### QA activity:
- [ ] Test plan has been approved by relevant developers
- [ ] Test plan has been peer reviewed by QA
- [ ] Cypress test cases have been added and approved by either SDET or
manual QA
- [ ] Organized project review call with relevant stakeholders after
Round 1/2 of QA
- [ ] Added Test Plan Approved label after reveiwing all Cypress test
2023-03-22 09:44:41 +05:30
Saroj
c4897157dd
ci: Deleting the combined-failed-spec-ci in ci-test-result (#21661)
## Description
- Deleting the combined-failed-spec-ci in ci-test-result

## Type of change
- Yml file changes

## How Has This Been Tested?
- Manual

## Checklist:
### QA activity:
- [ ] Test plan has been approved by relevant developers
- [ ] Test plan has been peer reviewed by QA
- [ ] Cypress test cases have been added and approved by either SDET or
manual QA
- [ ] Organized project review call with relevant stakeholders after
Round 1/2 of QA
- [ ] Added Test Plan Approved label after reveiwing all Cypress test
2023-03-22 00:23:31 +05:30
Saroj
3d4e0bc99d
ci: Deleting the failed-spec-ci once it is downloaded in ci-test-result to avoid getting specs from older runs (#21646)
## Description
- Deleting the failed-spec-ci once it is downloaded in ci-test-result to
avoid getting specs from older runs

## Type of change
- yml file changes


## How Has This Been Tested?
- Manual


## Checklist:
### QA activity:
- [ ] Test plan has been approved by relevant developers
- [ ] Test plan has been peer reviewed by QA
- [ ] Cypress test cases have been added and approved by either SDET or
manual QA
- [ ] Organized project review call with relevant stakeholders after
Round 1/2 of QA
- [ ] Added Test Plan Approved label after reveiwing all Cypress test
2023-03-21 21:23:10 +05:30
Saroj
b9952a2bd5
ci: Updated the combined_failed_spec_ci path in ci-test.yml (#21601)
## Description
- Updated the combined_failed_spec_ci path in ci-test.yml

## Type of change
- ci-test.yml

## How Has This Been Tested?
- Manual

## Checklist:
### QA activity:
- [ ] Test plan has been approved by relevant developers
- [ ] Test plan has been peer reviewed by QA
- [ ] Cypress test cases have been added and approved by either SDET or
manual QA
- [ ] Organized project review call with relevant stakeholders after
Round 1/2 of QA
- [ ] Added Test Plan Approved label after reveiwing all Cypress test
2023-03-20 18:34:17 +05:30
Saroj
b565f17ead
ci: Removing the old combined failed spec file (#21591)
## Description
- Removing the old combined failed spec file

## Type of change
- workflow files

## How Has This Been Tested?
- Manual

## Checklist:
### QA activity:
- [ ] Test plan has been approved by relevant developers
- [ ] Test plan has been peer reviewed by QA
- [ ] Cypress test cases have been added and approved by either SDET or
manual QA
- [ ] Organized project review call with relevant stakeholders after
Round 1/2 of QA
- [ ] Added Test Plan Approved label after reveiwing all Cypress test
2023-03-20 16:09:00 +05:30
Aishwarya-U-R
10015ec45e
ci: Adding AIRTABLE_BEARER to GitSecret (#21553)
## Description

- This PR added AIRTABLE_BEARER as GitHub secret to be used by CI runs

## Type of change

- Adding new token

## Checklist:
### QA activity:
- [X] Added Test Plan Approved label after reviewing all changes
2023-03-18 00:46:23 +05:30
Aishwarya-U-R
262630366a
ci: Stop restarting MsSQL container (#21460)
## Description

- This PR removes the restarting of the MsSQL container which is
throwing error sometimes in CI runs - WARNING: Failed to load
/var/opt/mssql/mssql.conf ini file with error open
/var/opt/mssql/mssql.conf: no such file or directory

## Type of change

- yml file update

## Checklist:
###  QA activity:
- [X] Added Test Plan Approved label after reviewing all changes
2023-03-16 12:00:36 +05:30
Aishwarya-U-R
6c0b8d053d
revert: Init Load into MSSQL container (#21456)
## Description

- This PR reverts the loading of data into MsSQL container with docker
exec - login timeout issue fixed.

## Type of change

- Yml file update (commenting two lines of code)


## Checklist:
### QA activity:
- [ ] Added Test Plan Approved label after reviewing all changes
2023-03-16 05:16:27 +05:30
Aishwarya-U-R
3ee9619e83
ci: Init loading data into MsSQL container (#21455)
## Description

- This PR init loads the data into MsSQL container after starting it

## Type of change

- Yml file update

## Checklist:
### QA activity:
- [ ] Added Test Plan Approved label after reviewing all changes
2023-03-16 04:29:31 +05:30
Aishwarya-U-R
fc4a09470e
test: Comment init loading into MsSQL container (#21454)
## Description

- This PR reverts the loading of data into MsSQL container with docker
exec, until JDBC driver issue fixed.

## Type of change

- Yml file update (commenting)

## Checklist:
### QA activity:
- [X] Added Test Plan Approved label after reviewing all changes
2023-03-16 04:06:21 +05:30
Saroj
7f559037af
test: Setup MSSQL docker container for CI runs (#21452)
## Description

- This PR includes MsSQL as separate docker container inside CI that can
be used for running mssql ds test cases in CI


## Type of change

- Yml file update

## How Has This Been Tested?

- CI-Debugging tmate session

## Checklist:
### QA activity:
- [X] Added Test Plan Approved label after reviewing all changes

---------

Co-authored-by: Aishwarya UR <aishwarya@appsmith.com>
2023-03-16 02:58:05 +05:30
Aishwarya-U-R
ed6d47f1af
Update ci-test.yml 2023-03-16 01:22:12 +05:30
Saroj
afceba00fe
ci: Picking up the unique records from the specs from the failed specs in ci-test.yml (#21450)
## Description
- Picking up the unique records from the specs from the failed specs in
ci-test.yml

## Type of change
- ci-test.yml

## How Has This Been Tested?
- Manual

## Checklist:
### QA activity:
- [ ] Test plan has been approved by relevant developers
- [ ] Test plan has been peer reviewed by QA
- [ ] Cypress test cases have been added and approved by either SDET or
manual QA
- [ ] Organized project review call with relevant stakeholders after
Round 1/2 of QA
- [ ] Added Test Plan Approved label after reveiwing all Cypress test
2023-03-16 00:39:19 +05:30
Aishwarya-U-R
fa5d70d273
Update ci-test.yml 2023-03-16 00:28:56 +05:30
Aishwarya-U-R
aaa5264a6d
test: Starting Mssql DB container for running MsSQL test scripts in CI runs (#21444)
## Description

- This PR includes MsSQL as separate docker container inside CI that can
be used for running mssql ds test cases in CI

## Type of change

- Yml file update

## Checklist:
### QA activity:
- [X] Added Test Plan Approved label after reviewing all changes

---------

Co-authored-by: Saroj <43822041+sarojsarab@users.noreply.github.com>
2023-03-15 23:36:54 +05:30
Saroj
8751d7925d
ci: Modified the logic to get the failed tests in ci-test.yml (#21430)
## Description
- Modified the logic to get the failed tests in ci-test.yml

## Type of change
- CI

## How Has This Been Tested?
- Manual

## Checklist:
### QA activity:
- [ ] Test plan has been approved by relevant developers
- [ ] Test plan has been peer reviewed by QA
- [ ] Cypress test cases have been added and approved by either SDET or
manual QA
- [ ] Organized project review call with relevant stakeholders after
Round 1/2 of QA
- [ ] Added Test Plan Approved label after reveiwing all Cypress test
2023-03-15 16:30:55 +05:30
Saroj
4ad92d648a
ci: CI optimisation to reduce ci-test run time and modularise docker image building (#21219)
## Description
- Modularised the docker image building
- Optimised the `install dependancies` step to reduce the time from 8
mins to 30-40 secs now
- Removed unnecessary steps from the ci-dubugging.yml 
- Added ci-debug.sh file to ease the steps to run ngrok while running
ci-debugging
- Changes made to below files
1.  integration-tests-command.yml
2. test-build-docker-image.yml
3. ci-test.yml
4. build-docker-image.yml

## Type of change
- integration-tests-command.yml
- test-build-docker-image.yml
- ci-test.yml
- build-docker-image.yml

## How Has This Been Tested?
- Manual

## Checklist:
### QA activity:
- [ ] Test plan has been approved by relevant developers
- [ ] Test plan has been peer reviewed by QA
- [ ] Cypress test cases have been added and approved by either SDET or
manual QA
- [ ] Organized project review call with relevant stakeholders after
Round 1/2 of QA
- [ ] Added Test Plan Approved label after reveiwing all Cypress test
2023-03-10 12:51:32 +05:30
Aishwarya-U-R
b592119746
test: Updating CI to use TED 6001 port for Oauth (#21265)
## Description

- This PR adds mapping for 6001 port from TED to run the OAuth cases in
CI

## Type of change

- Yml file update

## How Has This Been Tested?
- Local run with TED

## Checklist:
### QA activity:
- [ ] Added Test Plan Approved label after reviewing all changes
2023-03-08 18:30:36 +05:30
Aishwarya-U-R
e187b77a08
test: Fix to remove previous failing tests from Git Comment history (#21110)
## Description

- This PR updates the ci-test.yml file to empty the failed_spec_ci
folder which still holds the previoulsy (or attempt-1) failures & hence
prints the attempt-1 failures also in the attempt-2 git comment in PRs.

## Type of change
- ci-test.yml file update


## Checklist:
### QA activity:
- [X] Added Test Plan Approved label after reviewing all changes
2023-03-03 21:07:06 +05:30
Aishwarya-U-R
3f7756df6c
test: Change mapping port for GITEA to 3001 (#21093)
## Description

- This PR changes the mapping port of GITEA TED form 3000 to 3001
because while building client/server locally Appsmith uses
localhost:3000

## Type of change

- Script yml update (non-breaking change which fixes an issue)

## Checklist:
### QA activity:
- [X] Added Test Plan Approved label after reviewing the changes
2023-03-01 22:19:32 +05:30
Shrikant Sharat Kandula
94caa07a38 chore: Remove obsolete AuditLog env variables in CI 2023-02-26 06:20:27 +05:30
Saroj
bea69fa1d0
ci: updated the PAT to access the appsmithorg/ci-oldstack (#20870) 2023-02-22 16:22:44 +05:30
Saroj
32c04e3639
ci: Updated the pat to access appsmithorg/ci-oldstack (#20866)
## Description
- Updated the pat to access appsmithorg/ci-oldstack

## How Has This Been Tested?
- Cypress

## Checklist:
### QA activity:
- [ ] Test plan has been approved by relevant developers
- [ ] Test plan has been peer reviewed by QA
- [ ] Cypress test cases have been added and approved by either SDET or
manual QA
- [ ] Organized project review call with relevant stakeholders after
Round 1/2 of QA
- [ ] Added Test Plan Approved label after reveiwing all Cypress test
2023-02-22 15:29:51 +05:30
Saroj
200014965a
ci: Specifying the branch name to pull from ci-oldstack in ci-test.yml (#20863)
## Description
- Specifying the branch name to pull from ci-oldstack in ci-test.yml

## Type of change
- ci-test.yml

## How Has This Been Tested?
- Cypress

## Checklist:
### QA activity:
- [ ] Test plan has been approved by relevant developers
- [ ] Test plan has been peer reviewed by QA
- [ ] Cypress test cases have been added and approved by either SDET or
manual QA
- [ ] Organized project review call with relevant stakeholders after
Round 1/2 of QA
- [ ] Added Test Plan Approved label after reveiwing all Cypress test
2023-02-22 14:38:04 +05:30
Saroj
011e7b196d
ci: Download oldstack from private repo instead of S3 in ci-test.yml (#20802)
## Description
- Updated ci-test.yml to pull the old stack from a private repo instead
of S3

## Type of change
- ci-test.yml


## How Has This Been Tested?
- Cypress

## Checklist:
### QA activity:
- [ ] Test plan has been approved by relevant developers
- [ ] Test plan has been peer reviewed by QA
- [ ] Cypress test cases have been added and approved by either SDET or
manual QA
- [ ] Organized project review call with relevant stakeholders after
Round 1/2 of QA
- [ ] Added Test Plan Approved label after reveiwing all Cypress test
2023-02-22 12:56:40 +05:30
Trisha Anand
650ddf655c
ci : Preserve the docker logs across the parallel cypress runs to debug failures (#20671)
Co-authored-by: Aishwarya UR <aishwarya@appsmith.com>
2023-02-16 14:12:57 +05:30
Aishwarya-U-R
b9bc9eaf7d
test: TED support for GITEA (#20477)
## Description

- This PR Stops SSH which is run on port 22 for enabling it for Local
Gitea support from TED

## Type of change

- Ci-test yml file update:

## Checklist
### QA activity:
- [X] Added Test Plan Approved label after reviewing all changes
2023-02-14 22:59:40 +05:30
yatinappsmith
a3fb818c7c
ci: Disable telemetry in ci-test (#20614) 2023-02-14 15:45:59 +05:30
Goutham Pratapa
568544b5a3
ci: fix artifact names across the ci (#20586)
Fix: artifact names across CI
2023-02-13 14:45:02 +05:30
Aishwarya-U-R
3b36d86881
test: TED support for GITEA (stop ssh revert) (#20476)
## Description

- This PR reverts https://github.com/appsmithorg/appsmith/pull/20471

## Type of change
- ci-test yml file update

## Checklist:
### QA activity:
- [X] Added Test Plan Approved label after reviewing all changes
2023-02-08 12:50:38 +05:30
Aishwarya-U-R
3afb37e4ea
test: TED support for GITEA (stop ssh revert) (#20475)
## Description

This PR reverts https://github.com/appsmithorg/appsmith/pull/20471

## Type of change

- ci-test yml file update

## Checklist:
### QA activity:
- [X] Added Test Plan Approved label after reviewing all changes test
2023-02-08 12:44:05 +05:30
Aishwarya-U-R
3c9f68a033
test: TED support for GITEA (stop ssh revert) (#20473)
## Description
- This PR reverts #20471 

## Type of change

- ci-test yml file update

## Checklist:
### QA activity:
- [X] Added Test Plan Approved label after reveiwing all changes
2023-02-08 12:36:46 +05:30
Aishwarya-U-R
638c0d07fd
test: TED support for GITEA (#20471)
## Description

- This PR Stops SSH which is run on port 22 for enabling it for Local
Gitea support from TED


## Type of change

- Script update

## Checklist:
### QA activity:
- [X] Added Test Plan Approved label after reviewing the changes
2023-02-08 11:43:50 +05:30
Aishwarya-U-R
0d8a6e10f0
test: TED support for GITEA (#20456)
## Description

- This PR maps port 22 to 2222 (revert)

## Type of change

- Script fix (non-breaking change which fixes an issue)
2023-02-07 23:11:00 +05:30
Aishwarya-U-R
309a16d41c
test: TED support for GITEA (#20451)
## Description
- This PR adds the 3000 port mapping for the TED GITEA support for the
cypress tests


## Type of change

- Script update

## Checklist:
### QA activity:
- [X] Test plan has been peer reviewed by QA
- [X] Cypress test cases have been added and approved by either SDET or
manual QA
2023-02-07 21:44:07 +05:30
Parthvi
9e77cf95f1
ci: Add missing env variables for SAML and OIDC in workflow (#20382) 2023-02-04 14:41:30 +05:30
Saroj
f238c7b118
test: fix for ci-test skip issue (#20325)
## Description
- Updated the if condition to run the ci-test

## Type of change
- ci-test.yml


## How Has This Been Tested?
- Cypress

## Checklist:
### QA activity:
- [ ] Test plan has been approved by relevant developers
- [ ] Test plan has been peer reviewed by QA
- [ ] Cypress test cases have been added and approved by either SDET or manual QA
- [ ] Organized project review call with relevant stakeholders after Round 1/2 of QA
- [ ] Added Test Plan Approved label after reveiwing all Cypress test
2023-02-02 18:34:56 +05:30
Saroj
eb9f5280e9
ci: Modularize ci-test (#20240)
## Description
- Modularising the ci-test

## Type of change
- Workflow files

## Checklist:
### QA activity:
- [ ] Test plan has been approved by relevant developers
- [ ] Test plan has been peer reviewed by QA
- [ ] Cypress test cases have been added and approved by either SDET or
manual QA
- [ ] Organized project review call with relevant stakeholders after
Round 1/2 of QA
- [ ] Added Test Plan Approved label after reveiwing all Cypress test
2023-02-02 16:56:29 +05:30