Skip to content
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.

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

Feature: refactor pages and labels #41

Merged
merged 18 commits into from
Apr 17, 2017

Conversation

lipkau
Copy link
Member

@lipkau lipkau commented Apr 16, 2017

This implements:

lipkau and others added 13 commits April 15, 2017 23:44
* renamed function
* created custom object type for the set "Page+Labels"

implements AtlassianPS#23
* better handling of pipe input
* more tests for ContentLableSet objects
* Get-WikiLabel takes array of pages as input
* all functions return ContentLableSet

implements AtlassianPS#23
Must be alphabetical to match the check in ConfluencePS.Base.Tests.ps1
Copy link
Collaborator

@brianbunke brianbunke left a comment

Choose a reason for hiding this comment

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

Also changed this block in the integration tests for Add-WikiLabel

It 'label matches the specified value' {
    $NewLabel1.Labels.Name | Should BeExactly $Label1
    $NewLabel2.Labels.Name -contains $Label2 | Should Be $true
    $NewLabel3.Labels.Name -match $PartialLabel | Should Be ($Label1 + $Label2)
}

)

DynamicParam {
if (!($Page)) {
$paramDictionary["PageID"] | Out-String | Write-Host
Copy link
Collaborator

Choose a reason for hiding this comment

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

This block is triggering and causing the test errors on Get-WikiPage/Add-WikiLabel right now

Copy link
Member Author

Choose a reason for hiding this comment

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

removed with 48bdb98

I was experimenting with the parameters and forgot to remove this debug code

@brianbunke brianbunke merged commit b67aa0c into AtlassianPS:develop Apr 17, 2017
@lipkau lipkau added this to the v2.0 milestone Apr 18, 2017
@lipkau lipkau deleted the feature/refactorPagesAndLabels branch July 27, 2017 07:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants