Skip to content
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

Enhance light housefeature2 #1050

Merged
merged 45 commits into from
Apr 14, 2023

Conversation

Mohab-Osama
Copy link
Contributor

No description provided.

Mohab-Osama and others added 30 commits February 19, 2023 23:17
2- Add Test class to test the report generation
3- Add the Node JS script to the project folder
Add new TC
Edit DriverFactoryHelper
2- Edit the Test classes to test the report generation
3- Edit Node js Script
1- Add DriverFactoryHelper.java to use options
…into LHFixChanges

� Conflicts:
�	src/main/java/io/github/shafthq/shaft/driver/DriverFactoryHelper.java
1- Add AddNodeModules.sh file
…into UpdateScript3

� Conflicts:
�	AddNodeModules.sh
@codecov
Copy link

codecov bot commented Apr 12, 2023

Codecov Report

Merging #1050 (d7b5c06) into main (3dbf80e) will increase coverage by 16.00%.
The diff coverage is 6.66%.

@@              Coverage Diff              @@
##               main    #1050       +/-   ##
=============================================
+ Coverage     33.36%   49.36%   +16.00%     
- Complexity      760     1230      +470     
=============================================
  Files           106      106               
  Lines          9761     9806       +45     
  Branches       1004     1010        +6     
=============================================
+ Hits           3257     4841     +1584     
+ Misses         5936     4339     -1597     
- Partials        568      626       +58     
Impacted Files Coverage Δ
...performance/internal/LightHouseGenerateReport.java 0.00% <0.00%> (ø)
.../java/com/shaft/properties/internal/Reporting.java 94.59% <ø> (+55.70%) ⬆️
...com/shaft/driver/internal/DriverFactoryHelper.java 39.52% <50.00%> (+3.59%) ⬆️

... and 66 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

public void writeNodeScriptFileInProjectDirectory() {
List<String> commandsToServeLHReport;
if (SystemUtils.IS_OS_WINDOWS) {
commandsToServeLHReport = Arrays.asList("import puppeteer from 'puppeteer';\n" +
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Try using a multiline string to enhance this block

@MohabMohie MohabMohie merged commit 8aec50d into ShaftHQ:main Apr 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants