Skip to content

Comments

Delete old tests#3890

Merged
AndyScherzinger merged 1 commit intomasterfrom
deleteTests
Apr 12, 2019
Merged

Delete old tests#3890
AndyScherzinger merged 1 commit intomasterfrom
deleteTests

Conversation

@tobiasKaminsky
Copy link
Member

Just came across these old tests, which can be removed.

Signed-off-by: tobiasKaminsky tobias@kaminsky.me

Signed-off-by: tobiasKaminsky <tobias@kaminsky.me>
@nextcloud-android-bot
Copy link
Collaborator

Lint

TypemasterPR
Warnings5757
Errors00

SpotBugs (new)

Warning TypeNumber
Bad practice Warnings25
Correctness Warnings114
Internationalization Warnings15
Malicious code vulnerability Warnings4
Multithreaded correctness Warnings9
Performance Warnings124
Security Warnings48
Dodgy code Warnings131
Total470

SpotBugs (master)

Warning TypeNumber
Bad practice Warnings25
Correctness Warnings114
Internationalization Warnings15
Malicious code vulnerability Warnings4
Multithreaded correctness Warnings9
Performance Warnings124
Security Warnings48
Dodgy code Warnings131
Total470

@codecov
Copy link

codecov bot commented Apr 12, 2019

Codecov Report

Merging #3890 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##             master   #3890   +/-   ##
========================================
  Coverage      6.41%   6.41%           
  Complexity        1       1           
========================================
  Files           323     323           
  Lines         31220   31220           
  Branches       4477    4477           
========================================
  Hits           2003    2003           
  Misses        28921   28921           
  Partials        296     296

@AndyScherzinger AndyScherzinger merged commit 82fc476 into master Apr 12, 2019
@AndyScherzinger AndyScherzinger deleted the deleteTests branch April 12, 2019 20:03
tobiasKaminsky added a commit that referenced this pull request Apr 13, 2019
98a134f Merge pull request #3886 from nextcloud/ezaquarii/rename-preference-manager
75775ac Merge pull request #3871 from nextcloud/showCodacy
82fc476 Merge pull request #3890 from nextcloud/deleteTests
36937b2 Merge pull request #3882 from nextcloud/NPECheckScanner
1275728 Rename PreferenceManager to AppPreferencesImpl
0e17ebd delete old tests
1074bcf Merge pull request #3879 from nextcloud/pmdRuleset
09da127 - rename ruleset - add test_test to MainApp to test Codacy
8e1ebca daily dev 20190412
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants