Skip to content

tests: replace method withConsecutive #1025

Merged
saw-jan merged 5 commits into
release/2.11from
tests/replaceMethods
Jul 6, 2026
Merged

tests: replace method withConsecutive #1025
saw-jan merged 5 commits into
release/2.11from
tests/replaceMethods

Conversation

@nabim777

@nabim777 nabim777 commented May 26, 2026

Copy link
Copy Markdown
Collaborator

Description

This PR replaced the method withConsecutive with its alternatives according to its uses.

Related Issue or Workpackage

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Tests only (no source changes)

Checklist:

  • Code changes
  • Unit tests added
  • Acceptance tests added
  • Updated CHANGELOG.md file

@nabim777
nabim777 force-pushed the tests/replaceMethods branch 3 times, most recently from 62e1414 to 5e9dbd7 Compare June 1, 2026 03:47
@nabim777 nabim777 self-assigned this Jun 1, 2026
@nabim777
nabim777 force-pushed the tests/replaceMethods branch 2 times, most recently from c1aca76 to 7fc2c15 Compare June 5, 2026 03:50
@github-actions

Copy link
Copy Markdown

Hello there,
Thank you so much for taking the time and effort to create a pull request to our Nextcloud project.

We hope that the review process is going smooth and is helpful for you. We want to ensure your pull request is reviewed to your satisfaction. If you have a moment, our community management team would very much appreciate your feedback on your experience with this PR review process.

Your feedback is valuable to us as we continuously strive to improve our community developer experience. Please take a moment to complete our short survey by clicking on the following link: https://cloud.nextcloud.com/apps/forms/s/i9Ago4EQRZ7TWxjfmeEpPkf6

Thank you for contributing to Nextcloud and we hope to hear from you soon!

(If you believe you should not receive this message, you can add yourself to the blocklist.)

@nabim777
nabim777 force-pushed the tests/replaceMethods branch 4 times, most recently from b770ed9 to 725de5e Compare July 6, 2026 07:24
nabim777 added 5 commits July 6, 2026 14:50
Signed-off-by: nabim777 <nabinalemagar019@gmail.com>
Signed-off-by: nabim777 <nabinalemagar019@gmail.com>
Signed-off-by: nabim777 <nabinalemagar019@gmail.com>
Signed-off-by: nabim777 <nabinalemagar019@gmail.com>
Signed-off-by: nabim777 <nabinalemagar019@gmail.com>
@nabim777
nabim777 force-pushed the tests/replaceMethods branch from a5605d4 to dffb487 Compare July 6, 2026 09:05
@github-actions

github-actions Bot commented Jul 6, 2026

Copy link
Copy Markdown

JS Code Coverage

Coverage after merging tests/replaceMethods into release/2.11 will be
91.25%
Coverage Report
FileStmtsBranchesFuncsLinesUncovered Lines
src
   adminSettings.js0%0%0%0%1, 1, 10–19, 2–9
   bootstrap.js0%0%0%0%1, 1, 10–12, 2–9
   dashboard.js0%0%0%0%1, 1, 10–19, 2–9
   fileActions.js0%0%0%0%1, 1, 10–19, 2, 20–23, 3–9
   personalSettings.js0%0%0%0%1, 1, 10–19, 2–9
   projectTab.js0%0%0%0%1, 1, 10–19, 2, 20–29, 3, 30–39, 4, 40–49, 5, 50, 6–9
   reference.js0%0%0%0%1, 1, 10–19, 2, 20–29, 3, 30–39, 4, 40–45, 5–9
   utils.js85.92%33.33%50%87.59%12–20, 23–32
src/api
   endpoints.js100%100%100%100%
   settings.js59.09%100%0%68.42%10–11, 14–15, 18–19
src/components
   AdminSettings.vue96.05%94.26%96.08%96.31%448–453, 456–461, 527–528, 532–533, 536–537, 541–542, 552–557, 588–590, 609–611, 691–693, 727–729
   ErrorLabel.vue100%100%100%100%
   OAuthConnectButton.vue85.82%63.64%100%87.39%49–56, 64–69, 72–76
   PersonalSettings.vue92.02%95.65%90%91.71%133–134, 144–149, 152–161
src/components/admin
   FieldValue.vue100%100%100%100%
   FormAuthMethod.vue97.85%93.75%100%98.12%168, 222–224, 247–250
   FormHeading.vue100%100%100%100%
   FormOAuthSettings.vue97.61%96.15%93.10%98.10%217, 248, 338, 360, 378–380, 402–404
   FormOpenProjectHost.vue98.87%94.74%100%99.34%167–169, 279
   FormSSOSettings.vue98.57%97.18%96.55%98.91%231–233, 242–243, 358–359
   TermsOfServiceUnsigned.vue100%100%100%100%
   TextInput.vue100%100%100%100%
src/components/icons
   ClippyIcon.vue100%100%100%100%
   OpenProjectIcon.vue100%100%100%100%
src/components/settings
   CheckBox.vue100%100%100%100%
   ErrorNote.vue100%100%100%100%
   SettingsTitle.vue94.74%80%100%95.35%49–53
src/components/tab
   EmptyContent.vue96.45%80.95%100%98.24%102–105, 107–108, 97
   SearchInput.vue95.31%92.96%94.74%95.78%138–139, 192, 203–208, 267–269, 285–287, 291–296
   WorkPackage.vue86.18%72.50%93.33%87.58%106–115, 128–130, 141–145, 155–157, 175–181, 219, 219–224, 224, 224–235, 80–81
src/constants
   appID.js100%100%100%100%
   links.js100%100%100%100%
   messages.js100%100%100%100%
src/filesPlugin
   filesPlugin.js0%0%0%0%1, 1, 10, 100, 11–19, 2, 20–29, 3, 30–39, 4, 40–49, 5, 50–59, 6, 60–69, 7, 70–79, 8, 80–89, 9, 90–99
   filesPluginLessThan28.js0%0%0%0%1, 1, 10–19, 2, 20–29, 3, 30–39, 4, 40–49, 5, 50–59, 6, 60–69, 7, 70, 8–9
src/utils
   workpackageHelper.js93.80%93.10%88.89%94.24%100–102, 23–27, 54, 54–56, 97–99
src/views
   CreateWorkPackageModal.vue94.18%86.32%90.48%95.43%367–372, 375, 391, 508–511, 516–521, 526–531, 537–540, 543, 559, 559, 600–604, 614–616, 639–640, 648–650, 679–681, 703–705, 714–718
   Dashboard.vue92.96%92.86%82.61%93.77%120–125, 134, 144, 147, 158–160, 214–217, 220–221, 228–232, 67
   LinkMultipleFilesModal.vue99.14%97.56%100%99.32%157–159
   ProjectsTab.vue94.06%92.45%93.33%94.33%100–101, 107–109, 129, 140–141, 175–185, 234–236, 98–99

@github-actions

github-actions Bot commented Jul 6, 2026

Copy link
Copy Markdown

PHP Code Coverage

Coverage after merging tests/replaceMethods into release/2.11 will be
68.12%
Coverage Report
FileStmtsBranchesFuncsLinesUncovered Lines
integration_openproject/server/apps/integration_openproject/lib
   Capabilities.php0%100%0%0%24, 31–40
   OIDCClientMapper.php0%100%0%0%29, 40–49, 52
   TokenEventFactory.php100%100%100%100%
integration_openproject/server/apps/integration_openproject/lib/AppInfo
   Application.php36.59%100%50%35.53%115–117, 120–124, 126–131, 133–134, 137, 142–143, 145–148, 150, 152, 154, 158–161, 163–174, 176, 179, 183, 187–189, 222
integration_openproject/server/apps/integration_openproject/lib/BackgroundJob
   RemoveExpiredDirectUploadTokens.php0%100%0%0%28, 30–32, 41–42
integration_openproject/server/apps/integration_openproject/lib/Controller
   ConfigController.php79.01%100%70.59%79.36%114, 151–152, 154, 156–158, 160–163, 166–167, 169, 194, 198–200, 442–444, 446–448, 497, 577–581, 592, 606–609, 617, 621–624, 663, 669–684, 719, 722, 725–740, 754–757, 759–763
   DirectDownloadController.php0%100%0%0%33–35, 50–52, 54–61
   DirectUploadController.php71.03%100%100%70.21%117–119, 162–164, 175, 179–182, 184, 194, 201, 217–219, 221–222, 225–230, 233, 235, 245–247, 253–255, 263–265, 280–282, 301, 306, 312
   FilesController.php86.99%100%83.33%87.18%178–179, 241, 250, 267–270, 275–277, 282–284, 293
   OpenProjectAPIController.php81.09%100%82.35%80.98%116, 155, 202–204, 207–214, 216–220, 222, 241, 266, 331, 381, 401, 448, 473–475, 478–482, 484, 66
   OpenProjectController.php96.45%100%80%96.95%241–245
integration_openproject/server/apps/integration_openproject/lib/Dashboard
   OpenProjectWidget.php0%100%0%0%101, 108–109, 111–116, 118–122, 124–126, 129–140, 61–66, 73, 80, 87, 94
integration_openproject/server/apps/integration_openproject/lib/Exception
   OpenprojectAvatarErrorException.php100%100%100%100%
   OpenprojectErrorException.php100%100%100%100%
   OpenprojectFileNotUploadedException.php100%100%100%100%
   OpenprojectGroupfolderSetupConflictException.php100%100%100%100%
   OpenprojectResponseException.php100%100%100%100%
   OpenprojectUnauthorizedUserException.php0%100%0%0%21
integration_openproject/server/apps/integration_openproject/lib/Listener
   BeforeGroupDeletedListener.php100%100%100%100%
   BeforeNodeInsideOpenProjectGroupfilderChangedListener.php0%100%0%0%46–48, 52–55, 57, 59, 62–63, 65, 67–70, 72–75, 77–79
   BeforeUserDeletedListener.php0%100%0%0%30, 37–38, 40–43, 45
   LoadAdditionalScriptsListener.php0%100%0%0%37–38, 46–47, 49, 51–52, 54
   LoadSidebarScript.php65.91%100%100%64.29%105, 77–88, 95, 98
   OpenProjectReferenceListener.php0%100%0%0%45–47, 54–55, 57, 59–60, 62–74
   TermsOfServiceEventListener.php0%100%0%0%41–42, 47–48, 50–51, 53–55, 58–62
   UserChangedListener.php0%100%0%0%34, 41–42, 45–50, 53
integration_openproject/server/apps/integration_openproject/lib/Migration
   Version2001Date20221213083550.php0%100%0%0%30, 40–48, 50–58, 60–62, 64
   Version20100Date20250820101358.php0%100%0%0%38, 47–53, 56
   Version2310Date20230116153411.php0%100%0%0%29, 32–35, 37–62, 64–65, 67
   Version2400Date20230504144300.php0%100%0%0%30, 40–43
   Version2640Date20240628114301.php0%100%0%0%35, 47–49, 52–53, 56
   Version2900Date20250718065820.php0%100%0%0%22, 33–36, 38–39, 41–42, 45
integration_openproject/server/apps/integration_openproject/lib/Reference
   WorkPackageReferenceProvider.php51.67%100%25%58.33%100–101, 104, 108, 142, 150–151, 159, 37, 44, 51, 58–60, 87, 93–96, 99
integration_openproject/server/apps/integration_openproject/lib/Search
   OpenProjectSearchProvider.php0%100%0%0%100–102, 104–106, 109–110, 112–113, 116–125, 127–131, 52–55, 62, 69, 77, 79, 82, 89–90, 93–97, 99
   OpenProjectSearchResultEntry.php100%100%100%100%
integration_openproject/server/apps/integration_openproject/lib/Service
   DatabaseService.php42.31%100%60%40.43%108–112, 114, 63–76, 78–85
   DirectDownloadService.php88.46%100%100%87.50%62–63, 65
   DirectUploadService.php42.86%100%66.67%40%101, 62–65, 67–75, 95
   OauthService.php0%100%0%0%100–101, 40–42, 51–58, 60–66, 75–83, 94–99
   OpenProjectAPIService.php76.52%100%78.87%76.33%1005–1006, 1008, 1010–1011, 1034–1040, 1042–1049,

@nabim777 nabim777 changed the title tests: replace method withConsecutive tests: replace method withConsecutive Jul 6, 2026
@saw-jan
saw-jan merged commit 0363f10 into release/2.11 Jul 6, 2026
10 checks passed
@saw-jan
saw-jan deleted the tests/replaceMethods branch July 6, 2026 11:35
saw-jan added a commit that referenced this pull request Jul 7, 2026
* test(api): setup api test coverage for oidc method (#1050)

* test: separate test scenarios

Signed-off-by: Saw-jan <saw.jan.grg3e@gmail.com>

* test: disable ssl checks

Signed-off-by: Saw-jan <saw.jan.grg3e@gmail.com>

* test: add oidc setup API tests

Signed-off-by: Saw-jan <saw.jan.grg3e@gmail.com>

* test: cleanup team folders

Signed-off-by: Saw-jan <saw.jan.grg3e@gmail.com>

* test: fix scenarios

Signed-off-by: Saw-jan <saw.jan.grg3e@gmail.com>

* test: move teamfolder cleanup to relative context

Signed-off-by: Saw-jan <saw.jan.grg3e@gmail.com>

* test: teardown integration setup

Signed-off-by: Saw-jan <saw.jan.grg3e@gmail.com>

* chore: fix php code style

Signed-off-by: Saw-jan <saw.jan.grg3e@gmail.com>

* test: fix scenarios

Signed-off-by: Saw-jan <saw.jan.grg3e@gmail.com>

* test: fix scenario outline title

Signed-off-by: Saw-jan <saw.jan.grg3e@gmail.com>

* test: fix scenarios

Signed-off-by: Saw-jan <saw.jan.grg3e@gmail.com>

* test: use app password with OpenProject user

Signed-off-by: Saw-jan <saw.jan.grg3e@gmail.com>

* test: run teardown only if needed

Signed-off-by: Saw-jan <saw.jan.grg3e@gmail.com>

* test: fix scenario step indentation

Co-authored-by: Nalem7 <61624650+nabim777@users.noreply.github.com>
Signed-off-by: Sawjan Gurung <saw.jan.grg3e@gmail.com>

* test: teardown based on scenario tag

Signed-off-by: Saw-jan <saw.jan.grg3e@gmail.com>

* test: remove expected failure tag from passing scenarios

Signed-off-by: Saw-jan <saw.jan.grg3e@gmail.com>

* chore: address reviews

Signed-off-by: Saw-jan <saw.jan.grg3e@gmail.com>

* test: add expected failure tag

Signed-off-by: Saw-jan <saw.jan.grg3e@gmail.com>

---------

Signed-off-by: Saw-jan <saw.jan.grg3e@gmail.com>
Signed-off-by: Sawjan Gurung <saw.jan.grg3e@gmail.com>
Co-authored-by: Nalem7 <61624650+nabim777@users.noreply.github.com>

* tests: replace method `withConsecutive`  (#1025)

* test: replace method withConsecutive

Signed-off-by: nabim777 <nabinalemagar019@gmail.com>

* tes: use callback

Signed-off-by: nabim777 <nabinalemagar019@gmail.com>

* ci: fix some test

Signed-off-by: nabim777 <nabinalemagar019@gmail.com>

* fix the psalm error

Signed-off-by: nabim777 <nabinalemagar019@gmail.com>

* cleanup comments

Signed-off-by: nabim777 <nabinalemagar019@gmail.com>

---------

Signed-off-by: nabim777 <nabinalemagar019@gmail.com>

---------

Signed-off-by: Saw-jan <saw.jan.grg3e@gmail.com>
Signed-off-by: Sawjan Gurung <saw.jan.grg3e@gmail.com>
Signed-off-by: nabim777 <nabinalemagar019@gmail.com>
Co-authored-by: Nalem7 <61624650+nabim777@users.noreply.github.com>
Ashim-Stha pushed a commit that referenced this pull request Jul 7, 2026
* test: replace method withConsecutive

Signed-off-by: nabim777 <nabinalemagar019@gmail.com>

* tes: use callback

Signed-off-by: nabim777 <nabinalemagar019@gmail.com>

* ci: fix some test

Signed-off-by: nabim777 <nabinalemagar019@gmail.com>

* fix the psalm error

Signed-off-by: nabim777 <nabinalemagar019@gmail.com>

* cleanup comments

Signed-off-by: nabim777 <nabinalemagar019@gmail.com>

---------

Signed-off-by: nabim777 <nabinalemagar019@gmail.com>
saw-jan pushed a commit that referenced this pull request Jul 7, 2026
* test: replace method withConsecutive

Signed-off-by: nabim777 <nabinalemagar019@gmail.com>

* tes: use callback

Signed-off-by: nabim777 <nabinalemagar019@gmail.com>

* ci: fix some test

Signed-off-by: nabim777 <nabinalemagar019@gmail.com>

* fix the psalm error

Signed-off-by: nabim777 <nabinalemagar019@gmail.com>

* cleanup comments

Signed-off-by: nabim777 <nabinalemagar019@gmail.com>

---------

Signed-off-by: nabim777 <nabinalemagar019@gmail.com>
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.

2 participants