Skip to content

Removed test users env variables #1201

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

Merged
merged 1 commit into from
Mar 14, 2022
Merged

Removed test users env variables #1201

merged 1 commit into from
Mar 14, 2022

Conversation

jgebal
Copy link
Member

@jgebal jgebal commented Mar 13, 2022

Those variables are not needed and bring no real benefit asu the usernames actually cannot be configured and even if they were, there would be no real benefit to it.
Resolves: #1200

Those variables are not needed and bring no real benefit asu the usernames actually cannot be configured and even if they were, there would be no real benefit to it.
@jgebal jgebal added the bugfix label Mar 13, 2022
@jgebal jgebal added this to the 3.1.13 milestone Mar 13, 2022
@jgebal jgebal self-assigned this Mar 13, 2022
@github-actions
Copy link
Contributor

github-actions bot commented Mar 13, 2022

Unit Test Results

       1 files         2 suites   3m 44s ⏱️
1 008 tests 1 004 ✔️ 4 💤 0
1 011 runs  1 003 ✔️ 8 💤 0

Results for commit 61ce1bb.

♻️ This comment has been updated with latest results.

@codecov
Copy link

codecov bot commented Mar 13, 2022

Codecov Report

Merging #1201 (61ce1bb) into develop (ed93645) will increase coverage by 0.01%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop    #1201      +/-   ##
===========================================
+ Coverage    94.37%   94.39%   +0.01%     
===========================================
  Files          107      107              
  Lines         5063     5063              
  Branches        22       22              
===========================================
+ Hits          4778     4779       +1     
  Misses         277      277              
+ Partials         8        7       -1     
Flag Coverage Δ
11XE 93.76% <ø> (+0.06%) ⬆️
12.1EE 93.69% <ø> (+0.04%) ⬆️
18XE 94.02% <ø> (ø)
21XE 94.19% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
source/core/ut_utils.pkb 94.44% <0.00%> (+0.25%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 85fd36d...61ce1bb. Read the comment docs.

@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@jgebal jgebal merged commit 515c4bc into develop Mar 14, 2022
@jgebal jgebal deleted the feature/cleanup_env branch March 14, 2022 08:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Cannot install tests when using default user names as provided in template.env.sh
2 participants