content icon indicating copy to clipboard operation
content copied to clipboard

disable user QA

Open JudahSchwartz opened this issue 2 months ago • 3 comments

Status

  • [ ] In Progress
  • [x] Ready
  • [ ] In Hold - (Reason for hold)

Related Issues

fixes: CRTX-202222

Description

added some quick actions for platform chop of domain\samaccount name to samaccountname only - when we search by samaccoutnname in AD the domain isnt expected to show up, it is relevant in that its under that DN (which shuold be provided in the other parameter)

Must have

  • [ ] Tests
  • [ ] Documentation

JudahSchwartz avatar Nov 06 '25 10:11 JudahSchwartz

Coverage

Coverage Report
FileStmtsMissCoverMissing
Packs/Active_Directory_Query/Integrations/Active_Directory_Query
   Active_Directory_Query.py97759838%134–135, 137, 139–142, 150, 154–155, 200, 222–228, 230, 249–255, 257, 278–282, 284, 291–296, 303–306, 320–321, 324, 339–341, 350–352, 354–357, 359, 368–371, 373–375, 377, 393–401, 419–421, 425–428, 449, 484–486, 488–493, 497–502, 510, 517–519, 521, 525, 527–532, 534, 537–538, 540–543, 545, 554, 562, 570, 585, 596–597, 601–602, 606–607, 611–612, 616–620, 623, 641, 677–680, 682, 690, 692, 695–696, 698–699, 701–703, 707–710, 719, 721–724, 730–732, 735, 738–739, 742–743, 746–749, 753–757, 759–762, 766–767, 769–770, 781, 785, 807, 814, 822–823, 848–849, 851–852, 866–867, 869–874, 877–886, 889–894, 897, 899, 902–904, 907–909, 912, 918, 920, 925, 965, 974–975, 988, 992, 1042, 1046, 1057–1058, 1060–1061, 1066, 1076–1078, 1084, 1088, 1091–1093, 1107–1110, 1114–1115, 1117–1118, 1121–1127, 1130–1139, 1143–1145, 1147, 1152, 1156–1157, 1159–1166, 1168, 1171–1173, 1175–1176, 1186–1189, 1193, 1196–1200, 1202–1203, 1206, 1208, 1213, 1217, 1219–1223, 1225–1226, 1228, 1233, 1237, 1239–1241, 1244, 1246, 1251, 1255–1256, 1258–1259, 1261–1263, 1265, 1270, 1274–1275, 1277–1281, 1283, 1287, 1290–1292, 1294, 1300, 1302–1303, 1307–1308, 1310–1311, 1314–1317, 1320–1322, 1324–1325, 1352, 1391, 1394–1397, 1400–1401, 1403, 1408, 1417–1423, 1434–1437, 1443, 1445–1447, 1451–1454, 1456, 1459, 1461–1464, 1469, 1471–1475, 1477, 1486, 1488–1491, 1495, 1497, 1500–1501, 1503, 1505–1519, 1521–1523, 1526, 1528, 1543, 1546–1548, 1550, 1555, 1559, 1561, 1564–1565, 1567, 1569–1573, 1576, 1578, 1580–1582, 1586, 1591, 1595, 1598–1600, 1602–1604, 1606, 1611, 1619–1622, 1624, 1629, 1633–1634, 1636, 1639–1641, 1643–1644, 1648, 1650, 1652, 1654–1655, 1657, 1676–1678, 1680–1681, 1683–1686, 1689, 1691–1692, 1695–1696, 1698–1701, 1704–1706, 1709, 1725–1726, 1804–1806, 1827, 1840, 1842–1843, 1848–1849, 1851, 1855–1856, 1858–1859, 1861–1862, 1864–1865, 1867–1868, 1870–1871, 1873–1874, 1876–1877, 1879–1880, 1882–1883, 1885–1886, 1888–1889, 1891–1892, 1894–1895, 1897–1898, 1900–1901, 1903–1904, 1906–1907, 1909–1910, 1912–1913, 1915–1916, 1918–1919, 1921–1922, 1924–1925, 1927–1928, 1931–1933, 1935–1937, 1939–1941, 1943–1945, 1947–1949, 1952, 1954–1957, 1961–1962, 1967, 1969
TOTAL97759838% 

Tests Skipped Failures Errors Time
44 0 :zzz: 0 :x: 0 :fire: 11.211s :stopwatch:

github-actions[bot] avatar Nov 06 '25 10:11 github-actions[bot]

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? ✅

content-bot avatar Nov 23 '25 13:11 content-bot

⚠️ The PR is missing the ready-for-pipeline-running label. Please add the label when the PR is ready in order to proceed.

content-bot avatar Nov 24 '25 11:11 content-bot