Refactor the runningFullReset varaible

[SagarGi]

Description

The full reset of the integration means resetting all the values that has been set already default null. Some how the runningFullReset was resulting false even when we were resetting the whole integration. It was because of $oldClientSecret $oldClientId which made it false. These 2 checks has been removed since it seems to be non relevant to resetting the integration.

Related Issue or Workpackage

• Fixes

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

[github-actions]

JS Code Coverage

Coverage after merging fix/isFullResetVariable into master will be

86.86%

Coverage Report

File	Stmts	Branches	Funcs	Lines	Uncovered Lines
src
adminSettings.js	0%	0%	0%	0%	1, 1, 10–19, 2, 20–25, 3–9
bootstrap.js	0%	0%	0%	0%	1, 1–7
dashboard.js	0%	0%	0%	0%	1, 1, 10–19, 2, 20–25, 3–9
fileActions.js	0%	0%	0%	0%	1, 1, 10–17, 2–9
personalSettings.js	0%	0%	0%	0%	1, 1, 10–19, 2, 20–25, 3–9
projectTab.js	0%	0%	0%	0%	1, 1, 10–19, 2, 20–29, 3, 30–39, 4, 40–49, 5, 50–59, 6, 60–66, 7–9
reference.js	0%	0%	0%	0%	1, 1, 10–19, 2, 20–29, 3, 30–39, 4, 40–49, 5, 50–59, 6, 60, 7–9
utils.js	66.67%	33.33%	50%	69.09%	10–14, 17–26, 6–9
src/components
AdminSettings.vue	96.62%	21.05%	7.41%	99.91%	1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1, 1
OAuthConnectButton.vue	99.21%	87.50%	100%	100%	1
PersonalSettings.vue	98.87%	93.33%	85.71%	100%	1
src/components/admin
FieldValue.vue	97.12%	83.33%	100%	98.89%	1, 23, 23
FormHeading.vue	96.43%	44.44%	100%	99.36%	1, 1, 1, 1, 34, 34
ProjectFolderError.vue	96.72%	100%	50%	98.21%	1
TermsOfServiceUnsigned.vue	99.19%	90%	100%	100%	1
TextInput.vue	97.70%	93.75%	44.44%	100%	1
src/components/icons
ClippyIcon.vue	93.18%	50%	50%	97.50%	1, 1
OpenProjectIcon.vue	93.02%	50%	50%	97.44%	1, 1
src/components/settings
CheckBox.vue	92.45%	80%	66.67%	97.62%	1, 1
SettingsTitle.vue	91.11%	50%	100%	94.87%	1, 1, 10, 10
src/components/tab
EmptyContent.vue	96.09%	72.73%	100%	99.35%	1, 1, 1, 1, 1, 34, 34
SearchInput.vue	99.23%	94.74%	77.78%	100%	1
WorkPackage.vue	98.43%	40%	100%	99.36%	1, 1, 1, 43, 43
src/filesPlugin
filesPlugin.js	0%	0%	0%	0%	1, 1, 10, 100–104, 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.js	0%	0%	0%	0%	1, 1, 10–19, 2, 20–29, 3, 30–39, 4, 40–49, 5, 50–59, 6, 60–69, 7, 70–78, 8–9
src/utils
workpackageHelper.js	94.24%	93.48%	88.89%	94.68%	18–20, 49, 49–51, 94–99
src/views
CreateWorkPackageModal.vue	98.22%	92.59%	40%	99.84%	1, 1, 1
Dashboard.vue	98.97%	71.43%	100%	99.65%	1, 1, 1
LinkMultipleFilesModal.vue	99.67%	94.12%	100%	100%	1
ProjectsTab.vue	99.74%	93.75%	100%	100%	23
WorkPackagePickerElement.vue	0%	0%	0%	0%	1, 1, 10, 100–109, 11, 110–119, 12, 120–129, 13, 130–139, 14, 140–149, 15, 150–151, 16–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
WorkPackageReferenceWidget.vue	0%	0%	0%	0%	1, 1, 10, 100–109, 11, 110–119, 12, 120–129, 13, 130–134, 14–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

[github-actions]

PHP Code Coverage

Coverage after merging fix/isFullResetVariable into master will be

59.45%

Coverage Report

File	Stmts	Branches	Funcs	Lines	Uncovered Lines
server/apps/integration_openproject/lib
Capabilities.php	0%	100%	0%	0%	19, 26–29
server/apps/integration_openproject/lib/AppInfo
Application.php	10.53%	100%	25%	8.82%	103, 105–108, 110, 112, 114, 118–121, 123, 127–132, 143, 147, 74–76, 79, 83, 85, 88, 92, 94, 98
server/apps/integration_openproject/lib/BackgroundJob
RemoveExpiredDirectUploadTokens.php	0%	100%	0%	0%	42, 44–46, 55–56
server/apps/integration_openproject/lib/Controller
ConfigController.php	63.57%	100%	53.33%	64.15%	135, 152–153, 155, 157–159, 161–162, 167–168, 170, 212, 221, 282, 371–372, 375–376, 382–384, 424, 512–515, 517–518, 521, 529, 540, 554–556, 571–575, 577–578, 580–583, 585–587, 589–591, 593–595, 613–618, 620–621, 623–625, 628, 630–633, 635–637, 639–641, 643–645, 659, 668–671, 673–676, 686–691
DirectDownloadController.php	0%	100%	0%	0%	36–38, 53–55, 57, 60–61
DirectUploadController.php	71.43%	100%	100%	70.43%	131–132, 177, 190, 194–197, 199, 209, 216, 232–234, 236–237, 240–242, 248, 250, 255–256, 263–264, 267–268, 271–272, 288–289, 309, 314, 320
FilesController.php	73.86%	100%	83.33%	73.17%	181–182, 244, 250, 252–254, 256, 258, 269–271, 274–275, 277–278, 282–285, 288, 294
OpenProjectAPIController.php	83.87%	100%	80%	84.17%	139, 180, 204, 230–233, 236, 238, 240–243, 245–246, 251, 263, 272, 290, 299, 367, 369, 419, 421, 441, 443, 490, 492, 518–521, 524–528, 530, 735, 737, 96
server/apps/integration_openproject/lib/Dashboard
OpenProjectWidget.php	0%	100%	0%	0%	101–102, 104–108, 116, 123–124, 126, 128–129, 131–132, 134, 137–138, 140–141, 143, 69–73, 80, 87, 94
server/apps/integration_openproject/lib/Exception
OpenprojectErrorException.php	100%	100%	100%	100%
OpenprojectFileNotUploadedException.php	100%	100%	100%	100%
OpenprojectGroupfolderSetupConflictException.php	100%	100%	100%	100%
OpenprojectResponseException.php	100%	100%	100%	100%
OpenprojectUnauthorizedUserException.php	0%	100%	0%	0%	16
server/apps/integration_openproject/lib/Listener
BeforeGroupDeletedListener.php	0%	100%	0%	0%	48, 56–57, 60–63
BeforeNodeInsideOpenProjectGroupfilderChangedListener.php	0%	100%	0%	0%	41–43, 47–52, 54, 57–58, 60–65, 67, 69, 73
BeforeUserDeletedListener.php	0%	100%	0%	0%	48, 55–56, 58–61
LoadAdditionalScriptsListener.php	0%	100%	0%	0%	17–18, 21–25
LoadSidebarScript.php	0%	100%	0%	0%	100, 102, 104, 106–108, 110, 112, 114–115, 117–118, 120, 122, 75–81, 83–84, 86–87, 89–90, 96–97, 99
OpenProjectReferenceListener.php	0%	100%	0%	0%	53–54, 58–59, 62–63, 65, 67
TermsOfServiceEventListener.php	0%	100%	0%	0%	59–60, 64–66, 68–69, 71–73, 76–78
UserChangedListener.php	0%	100%	0%	0%	52, 59–60, 63–68
server/apps/integration_openproject/lib/Migration
Version2001Date20221213083550.php	0%	100%	0%	0%	47, 57, 60, 63, 67, 70, 73, 77–79, 81
Version2310Date20230116153411.php	0%	100%	0%	0%	46, 49–52, 54–56, 60, 64, 68, 72, 76, 81–82, 84
Version2400Date20230504144300.php	0%	100%	0%	0%	47, 57, 60
server/apps/integration_openproject/lib/Reference
WorkPackageReferenceProvider.php	44%	100%	18.18%	51.28%	102, 108–111, 114–116, 119, 123, 157, 165–166, 174, 52, 59, 66, 73–74
server/apps/integration_openproject/lib/Search
OpenProjectSearchProvider.php	0%	100%	0%	0%	103–104, 107–114, 116–118, 121–122, 124–125, 129–134, 140–141, 143, 66–69, 76, 83, 91, 93, 96
OpenProjectSearchResultEntry.php	100%	100%	100%	100%
server/apps/integration_openproject/lib/Service
DatabaseService.php	43.90%	100%	60%	41.67%	125–128, 131, 80–87, 89–93, 95–97
DirectDownloadService.php	88%	100%	100%	86.96%	65–66, 68
DirectUploadService.php	54.55%	100%	66.67%	52.63%	112, 118, 79–82, 84, 89, 91
OauthService.php	0%	100%	0%	0%	108–115, 45–47, 56–66, 68, 70–72, 75–78, 89, 91–94, 96–97
OpenProjectAPIService.php	75.30%	100%	75%	75.33%	1001–1003, 1005–1007, 1010, 1028, 1030,

[individual-it]

some tests would be good if they make sense

[SagarGi]

some tests would be good if they make sense

We have tests for resetting. But the i think we cannot test for that single variable.