Skip to content

Conversation

itssapir
Copy link
Contributor

Status

  • In Progress
  • Ready
  • In Hold - (Reason for hold)

Related Issues

related:
XSUP-56831
XSUP-57143
XSUP-57411
XSUP-55886

Description

Improve error messaging in the test module and from commands for Jira Cloud instances that were configured as On-Prem.

Must have

  • Tests
  • Documentation

@itssapir itssapir requested a review from omerKarkKatz October 19, 2025 09:14
@itssapir itssapir added the ready-for-pipeline-running Whether the pr is ready for running the whole pipeline, including testing on SAAS machines label Oct 19, 2025
@github-actions
Copy link

github-actions bot commented Oct 19, 2025

Coverage

Coverage Report
FileStmtsMissCoverMissing
Packs/Jira/Integrations/JiraV3
   JiraV3.py152028781%164, 167, 192–193, 195–198, 202–207, 211–214, 291, 296, 314–315, 332–333, 351–352, 364, 393, 396, 407, 426–427, 451–452, 466, 480, 493, 510, 526–527, 543, 559–560, 578–579, 597–598, 616, 632–633, 652, 654, 675, 677, 693, 709, 723, 739–740, 750, 764, 779, 782, 798, 809, 853–854, 860–861, 863, 865, 920, 923, 926, 942, 950, 953–955, 958, 962–963, 966, 968, 972, 974, 982, 988–989, 996–997, 1009, 1019–1020, 1032, 1050, 1058, 1066–1067, 1103, 1121–1126, 1132, 1135–1137, 1145, 1148–1150, 1153, 1156, 1160–1161, 1164, 1166, 1170–1171, 1174–1175, 1184, 1190, 1197–1198, 1201, 1204, 1213–1214, 1222, 1232, 1238–1241, 1243–1244, 1462–1475, 1558, 1594–1595, 1619, 1655–1657, 1728, 1748, 1785–1786, 1789–1790, 1806, 1810, 1813–1815, 1817–1818, 1820, 1935, 1967, 1982–1983, 2053, 2055, 2086, 2103, 2130, 2138, 2289, 2354, 2357–2361, 2388, 2401, 2565, 2628, 2643, 2648, 2699, 2715, 2741, 2861, 2972, 3059, 3079, 3116, 3137, 3148, 3165, 3488, 3502–3503, 3524–3526, 3544–3545, 3557, 3561, 3826–3827, 3862, 3868, 3875, 3882–3883, 4020–4024, 4139, 4142, 4158, 4179–4180, 4196–4200, 4237–4238, 4319, 4323–4324, 4326–4327, 4329–4331, 4333–4334, 4422, 4424, 4481, 4489, 4527, 4540, 4603–4607, 4609, 4625–4626, 4628–4629, 4635, 4637–4639, 4641, 4680–4682, 4693–4694, 4718, 4776–4777, 4793–4794, 4802–4803, 4812
TOTAL152028781% 

Tests Skipped Failures Errors Time
180 0 💤 0 ❌ 0 🔥 3.817s ⏱️

@itssapir itssapir marked this pull request as ready for review October 19, 2025 14:22
@itssapir itssapir requested a review from dantavori as a code owner October 19, 2025 14:22
Copy link
Contributor

@omerKarkKatz omerKarkKatz left a comment

Choose a reason for hiding this comment

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

Nice!

@richardbluestone richardbluestone requested review from talihaff and removed request for richardbluestone October 22, 2025 06:53
@talihaff
Copy link
Contributor

docs approved, see comments

@itssapir itssapir enabled auto-merge (squash) October 22, 2025 12:35
@content-bot
Copy link
Collaborator

Validate summary
The following errors were thrown as a part of this pr: .
If the AG100 validation in the pre-commit GitHub Action fails, the pull request cannot be force-merged.

Verdict: PR can be force merged from validate perspective? ✅

@itssapir itssapir merged commit 23d6802 into master Oct 22, 2025
18 of 19 checks passed
@itssapir itssapir deleted the jira-improve-error-messaging branch October 22, 2025 15:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs-approved ready-for-pipeline-running Whether the pr is ready for running the whole pipeline, including testing on SAAS machines

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants