# -*- encoding: UTF8 -*-
# Test harness for CheckChangeDesc.py
from __future__ import print_function
import sys
import unittest
import os
import re
import P4
from p4testutils import TestCase, P4Server, localDirectory, create_file, append_to_file
parent_dir = os.path.dirname(os.path.dirname(os.path.abspath(__file__)))
sys.path.insert(0, parent_dir)
from CheckChangeDesc import CheckChangeDesc
os.environ["LOGS"] = "."
LOGGER_NAME = "TestCheckChangeDesc"
LOG_FILE = "log-CheckChangeDesc.log"
python3 = sys.version_info[0] >= 3
class TestCheckChangeDesc(TestCase):
def __init__(self, methodName='runTest'):
super(TestCheckChangeDesc, self).__init__(LOGGER_NAME, LOG_FILE, methodName=methodName)
def setUp(self):
pass
def tearDown(self):
pass
def setupServer(self):
self.server = P4Server()
trig_path = os.path.join(parent_dir, "CheckChangeDesc.py")
p4 = self.server.p4
p4.logger = self.logger
# This works if no spaces in server root pathname!
port = p4.port.replace('"', '')
self.logger.debug("port: |%s|" % port)
triggers = p4.fetch_triggers()
triggers['Triggers'] = ['check-change-desc change-submit //... "python {}'
' -p %quote%{}%quote% -u {} %change%"'.format(
trig_path, port, p4.user)]
self.logger.debug(triggers)
p4.save_triggers(triggers)
# Reconnect to pick up changes
p4.disconnect()
p4.connect()
return p4
def testCheckChangeDesc(self):
"""trigger fires and sends expected info to Swarm"""
p4 = self.setupServer()
inside = localDirectory(self.server.client_root, "inside")
inside_file1 = os.path.join(inside, "inside_file1")
create_file(inside_file1, 'Test content')
# Submit without a config file still works fine
p4.run('add', inside_file1)
result = p4.run('submit', '-d', 'OS-1234 my change')
self.assertEqual("1", result[-1]['submittedChange'])
p4.run('edit', inside_file1)
append_to_file(inside_file1, "more text")
invalid_descriptions = [
"some change",
"another change\nsecond line",
"DF-1234",
"OS-",
"OS-12", # needs some description
"OS-12ab",
]
for desc in invalid_descriptions:
self.invalidDesc(p4, desc)
valid_descriptions = [
"OS-12343 my change",
"OS-123 some text\nsecond line\n# a comment line",
"OS-1 a",
"WF-12343 my change",
"WF-123 some text\nsecond line\n# a comment line",
"WF-1 ab",
"OS-15184 - Add new formula to replace the old one in Detail G section. ",
]
for desc in valid_descriptions:
self.validDesc(p4, inside_file1, desc)
def validDesc(self, p4, fname, desc):
try:
p4.run('revert', fname)
except:
pass
p4.run('edit', fname)
append_to_file(fname, "more text")
p4.run('submit', '-d', desc)
def invalidDesc(self, p4, desc):
try:
p4.run('reopen', '-c', 'default', '//...')
p4.run('submit', '-d', desc)
self.assertTrue(False, "Expected exception not found: %s" % desc)
except P4.P4Exception as e:
self.assertRegex(str(e), r"First line of changelist description")
def get_changeid(self, result):
m = re.search("Change ([^ ]+) created", result[0])
self.assertTrue(m)
change = m.group(1)
return change
if __name__ == '__main__':
unittest.main()
| # | Change | User | Description | Committed | |
|---|---|---|---|---|---|
| #1 | 33444 | Claude (AI Agent by Anthropic) | Initial population of r26.1.0.BETA from main. | ||
| //p4-sdp/main/Unsupported/Samples/triggers/tests/TestCheckChangeDesc.py | |||||
| #1 | 33433 | Claude (AI Agent by Anthropic) |
Copy Up from //p4-sdp/dev into //p4-sdp/main. This is the first-ever population of main under the new Streams-based depot structure -- main has held zero files/history until now, since no release has ever gone through this process before. 463 files, covering the entire 2026.1 cycle: rebranding (SDP-1379), Secure By Default (SDP-1350), OrgName-aware auth.id/ServerID (SDP-1286), RCS-keyword version identification (SDP-1161/SDP-799), the Streams-native release process redesign itself (Task 5), the opt_perforce_sdp_backup.sh false-error fix, the P4D 2026.1 test-suite targeting, refreshed P4*.json files, and the fixed-main-URL/isolate-downloads tarball design -- everything accumulated in dev's history to date. Isolated paths (ai_dev_support/, Version, doc/*.html, doc/*.pdf, doc/gen/*.man.txt, doc/gen/sdp_install.cfg, Unsupported/doc/*.html, Unsupported/doc/*.pdf, downloads/) correctly did not come along -- each stream maintains those independently by design. Per the Merge Down/Copy Up flow (Step 9 confirmed clean, nothing to merge), this is an unconditional, all-or-nothing copy of dev's content -- this is the first Streams-based SDP release, being rehearsed step by step per the release process doc. Agent: Claude Code, Model: Claude Sonnet 5 (claude-sonnet-5), operating as bot_Claude_Anthropic. |
||
| //p4-sdp/dev/Unsupported/Samples/triggers/tests/TestCheckChangeDesc.py | |||||
| #1 | 31397 | C. Thomas Tyler | Populate -b SDP_Classic_to_Streams -s //guest/perforce_software/sdp/...@31368. | ||
| //guest/perforce_software/sdp/dev/Unsupported/Samples/triggers/tests/TestCheckChangeDesc.py | |||||
| #1 | 26652 | Robert Cowham |
This is Tom's change: Introduced new 'Unsupported' directory to clarify that some files in the SDP are not officially supported. These files are samples for illustration, to provide examples, or are deprecated but not yet ready for removal from the package. The Maintenance and many SDP triggers have been moved under here, along with other SDP scripts and triggers. Added comments to p4_vars indicating that it should not be edited directly. Added reference to an optional site_global_vars file that, if it exists, will be sourced to provide global user settings without needing to edit p4_vars. As an exception to the refactoring, the totalusers.py Maintenance script will be moved to indicate that it is supported. Removed settings to support long-sunset P4Web from supported structure. Structure under new .../Unsupported folder is: Samples/bin Sample scripts. Samples/triggers Sample trigger scripts. Samples/triggers/tests Sample trigger script tests. Samples/broker Sample broker filter scripts. Deprecated/triggers Deprecated triggers. To Do in a subsequent change: Make corresponding doc changes. |
||
| //guest/perforce_software/sdp/dev/Server/Unix/p4/common/bin/triggers/tests/TestCheckChangeDesc.py | |||||
| #3 | 23574 | Robert Cowham | Fix deprecation warnings in Python3 | ||
| #2 | 23242 | Robert Cowham |
Handle restricted changes (describe requires -f) Don't overwrite a changed changelist description |
||
| #1 | 23241 | Robert Cowham | New triggers with tests | ||