Skip to content

[DEVX-5163] Add repository_url field to upstream:list command #2725

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
namespacebrian wants to merge 4 commits into
base: 4.x
Choose a base branch
from
Open

[DEVX-5163] Add repository_url field to upstream:list command #2725

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
233c8e5
Back to dev
namespacebrian Sep 11, 2025
b4ea33f
Add repository_url field to upstream:list command
namespacebrian Sep 19, 2025
464002b
Exclude repository_url from upstream:list default fields
namespacebrian Sep 19, 2025
df51b82
Fix code style: break long lines in test assertions
namespacebrian Sep 20, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 2 additions & 0 deletions CHANGELOG.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,6 +1,8 @@
# Change Log
All notable changes to this project will be documented in this file. This project adheres to [Semantic Versioning](http://semver.org)

## 4.0.4-dev

## 4.0.3 - 2025-09-11

### Added
Expand Down
2 changes: 1 addition & 1 deletion config/constants.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@
---

# App
TERMINUS_VERSION: '4.0.3'
TERMINUS_VERSION: '4.0.4-dev'

# Connectivity
TERMINUS_HOST: 'terminus.pantheon.io'
Expand Down
2 changes: 2 additions & 0 deletions src/Commands/Upstream/ListCommand.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,9 @@ class ListCommand extends TerminusCommand
* category: Category
* type: Type
* framework: Framework
* repository_url: Repository URL
* organization: Organization
* @default-fields id,label,machine_name,category,type,framework,organization
* @option all Show all upstreams
* @option framework DEPRECATED Framework filter
* @option name DEPRECATED Name filter
Expand Down
28 changes: 28 additions & 0 deletions tests/Functional/UpstreamCommandsTest.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -27,6 +27,34 @@ public function testUpstreamListCommand()
$this->assertArrayHasKey('label', $upstreamInfo, 'An upstream should have "label" field.');
$this->assertArrayHasKey('machine_name', $upstreamInfo, 'An upstream should have "machine_name" field.');
$this->assertArrayHasKey('type', $upstreamInfo, 'An upstream should have "type" field.');
$this->assertArrayNotHasKey(
'repository_url',
$upstreamInfo,
'An upstream should not have "repository_url" field by default.'
);
}

/**
* Test UpstreamListCommand with repository_url field
*
* @test
* @covers \Pantheon\Terminus\Commands\Upstream\ListCommand
*
* @group upstream
* @group short
*/
public function testUpstreamListCommandWithRepositoryUrl()
{
$upstreamList = $this->terminusJsonResponse('upstream:list --fields=id,label,repository_url');
$this->assertIsArray($upstreamList);
$upstreamInfo = array_shift($upstreamList);
$this->assertArrayHasKey('id', $upstreamInfo, 'An upstream should have "id" field.');
$this->assertArrayHasKey('label', $upstreamInfo, 'An upstream should have "label" field.');
$this->assertArrayHasKey(
'repository_url',
$upstreamInfo,
'An upstream should have "repository_url" field when explicitly requested.'
);
}

/**
Expand Down
Loading