Added tests for hashtabletoPSCustomObjectConversion, OutErrorVairable… by bingbing8 · Pull Request #2160 · PowerShell/PowerShell

Yanbing (bingbing8) · 2016-09-01T21:58:27Z

Added tests for hashtabletoPSCustomObjectConversion, OutErrorVairable. Please review.

… tests.

Microsoft Contribution License Agreements (msftclas) · 2016-09-01T21:58:30Z

Hi Yanbing (@bingbing8), I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

It looks like you're a Microsoft contributor (Yanbing Wang). If you're full-time, we DON'T require a Contribution License Agreement. If you are a vendor, please DO sign the electronic Contribution License Agreement. It will take 2 minutes and there's no faxing! https://cla.microsoft.com.

TTYL, MSBOT;

Microsoft Contribution License Agreements (msftclas) · 2016-09-02T00:04:17Z

Hi Yanbing (@bingbing8), I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

It looks like you're a Microsoft contributor (Yanbing Wang). If you're full-time, we DON'T require a Contribution License Agreement. If you are a vendor, please DO sign the electronic Contribution License Agreement. It will take 2 minutes and there's no faxing! https://cla.microsoft.com.

TTYL, MSBOT;

James Truher (JamesWTruher) · 2016-09-02T00:08:53Z

��D�e�s�c�r�i�b�e� �"�T�e�s�t�s� �f�o�r� �h�a�s�h�t�a�b�l�e� �t�o� �P�S�C�u�s�t�o�m�O�b�j�e�c�t� �c�o�n�v�e�r�s�i�o�n�"� �-�T�a�g�s� �"�C�I�"� �{�

this file looks to be the wrong encoding - can you be sure that the BOM is not present and that the file is UTF8 (or ascii) - it's really hard to read

Refers to: test/powershell/Language/Scripting/HashtableToPSCustomObjectConversion.Tests.ps1:1 in 31f22c6. [](commit_id = 31f22c6, deletion_comment = False)

James Truher (JamesWTruher) · 2016-09-02T00:12:15Z

� � � � � � � � �I�t� �'�$�a� �t�y�p�e�'� �{� �$�a� �-�i�s� �[�S�y�s�t�e�m�.�M�a�n�a�g�e�m�e�n�t�.�A�u�t�o�m�a�t�i�o�n�.�P�S�O�b�j�e�c�t�]� �|� �S�h�o�u�l�d� �B�e� �$�t�r�u�e� �}� � � � � � � � �

this can be

it '$a type' { $a | should BeOfType "System.Management.automation.psobject" }

Refers to: test/powershell/Language/Scripting/HashtableToPSCustomObjectConversion.Tests.ps1:15 in 31f22c6. [](commit_id = 31f22c6, deletion_comment = False)

James Truher (JamesWTruher) · 2016-09-02T00:27:38Z

�

I don't think we really need a new type here - wouldn't this be ok?

describe "good description here" {
    it "good test name here" {
        try {
            new-object System.Management.Automation.MetadataException -property @{ Source = "foobar"; aa = 1 }
            throw "OK"
        }
        catch {
            $_.Exception | Should BeOfType "InvalidOperationException"
            $_.Exception.Message | Should Match "aa"
            $_.FullyQualifiedErrorId | should be "InvalidOperationException,Microsoft.PowerShell.Commands.NewObjectCommand"
        }
    }
}

Refers to: test/powershell/Language/Scripting/HashtableToPSCustomObjectConversion.Tests.ps1:66 in 31f22c6. [](commit_id = 31f22c6, deletion_comment = False)

James Truher (JamesWTruher) · 2016-09-02T00:37:10Z

+    }
+
+    Context 'Updating OutVariable Case 1: pipe string' {        
+


These aren't going to do anything because they're not in it blocks, also these really look like they could all be done via test cases

$testcases = @{ Title = "Updating OutVariable Case 1: pipe string"; command = "get-foo1"; OutVariable = "a"; Expected = "foo" }, @{ Title = 'Updating OutVariable Case 2: $pscmdlet.writeobject'; command = "get-foo1"; OutVariable = "a"; Expected = "foo" } Describe Tests { BeforeAll { function get-foo1 { [CmdletBinding()] param() "foo" } function get-foo2 { [CmdletBinding()] param() $pscmdlet.writeobject("foo") } function get-bar { [CmdletBinding()] param() "bar" get-foo1 -outVariable global:a } } It "<Title>" -TestCases $testcases { param ( $command, $arguments, $Expected, $OutVariable ) & $command -OutVariable $OutVariable > $null Get-Variable -ValueOnly $outVariable | should be $Expected } }

fixed

In reply to: 77278055 [](ancestors = 77278055)

Yanbing (bingbing8) · 2016-09-02T21:28:19Z

��D�e�s�c�r�i�b�e� �"�T�e�s�t�s� �f�o�r� �h�a�s�h�t�a�b�l�e� �t�o� �P�S�C�u�s�t�o�m�O�b�j�e�c�t� �c�o�n�v�e�r�s�i�o�n�"� �-�T�a�g�s� �"�C�I�"� �{�

fixed

In reply to: 244248919 [](ancestors = 244248919)

Refers to: test/powershell/Language/Scripting/HashtableToPSCustomObjectConversion.Tests.ps1:1 in 31f22c6. [](commit_id = 31f22c6, deletion_comment = False)

Yanbing (bingbing8) · 2016-09-02T21:28:26Z

� � � � � � � � �I�t� �'�$�a� �t�y�p�e�'� �{� �$�a� �-�i�s� �[�S�y�s�t�e�m�.�M�a�n�a�g�e�m�e�n�t�.�A�u�t�o�m�a�t�i�o�n�.�P�S�O�b�j�e�c�t�]� �|� �S�h�o�u�l�d� �B�e� �$�t�r�u�e� �}� � � � � � � � �

fixed

In reply to: 244249354 [](ancestors = 244249354)

Refers to: test/powershell/Language/Scripting/HashtableToPSCustomObjectConversion.Tests.ps1:15 in 31f22c6. [](commit_id = 31f22c6, deletion_comment = False)

Yanbing (bingbing8) · 2016-09-02T21:34:41Z

�

fixed

In reply to: 244251597 [](ancestors = 244251597)

Refers to: test/powershell/Language/Scripting/HashtableToPSCustomObjectConversion.Tests.ps1:66 in 31f22c6. [](commit_id = 31f22c6, deletion_comment = False)

… o scipt scope.

Yanbing (bingbing8) · 2016-09-07T20:10:41Z

James Truher (@JamesWTruher) I updated the tests based on your feedback. some of the replies don't show in browser. please check out. Thanks!

sergei (vors) · 2016-09-08T20:39:38Z

Assigning the PR to James Truher (@JamesWTruher)

James Truher (JamesWTruher) · 2016-09-09T21:43:33Z

+
+        It '$a should not be $null' { $script:a | Should Not Be $null }
+        It '$a type' { $script:a | should BeOfType "System.Management.automation.psobject" }
+    }


it seems that this is much too complicated, can't it just be:

It 'New-Object cmdlet should accept empty hashtable or $null as Property argument' { $a = new-object psobject -property $a | should not BeNullOrEmpty $a | should BeOfType "System.Management.Automation.PSObject" }

I don't see the point of the extra it statements, if the first line throws the test will fail

fixed.

In reply to: 78253881 [](ancestors = 78253881)

James Truher (JamesWTruher) · 2016-09-09T22:09:37Z

Yanbing (bingbing8) · 2016-09-12T01:22:00Z

James Truher (@JamesWTruher) Thanks for your feedback. I addressed all of them. Please check.

James Truher (JamesWTruher) · 2016-09-12T18:22:56Z

sergei (vors) · 2016-09-12T18:43:06Z

@@ -0,0 +1,10 @@
+<?xml version="1.0" encoding="utf-8" standalone="no"?>


please, remove this file from the PR.
In #2182 Douglas Thrift (@douglaswth) added it to ,gitignore.

Particularly because this file contains env-specific line

<NuGetPackageRoot>C:\git\part4\Packages</NuGetPackageRoot>

Sergei (@sergei) Vorobev good catch. I removed the file. Thanks!

In reply to: 78430315 [](ancestors = 78430315)

sergei (vors) · 2016-09-12T18:44:28Z

🕐

… tests (PowerShell#2160)

Added tests for hashtabletoPSCustomObjectConversion, OutErrorVairable…

96ff914

… tests.

Microsoft Contribution License Agreements (msftclas) added the cla-not-required label Sep 1, 2016

Update the title of hashtable to PSCustomObject conversion.

31f22c6

Travis Plunk (TravisEz13) added the Review - Needed label Sep 2, 2016

sergei (vors) closed this Sep 2, 2016

sergei (vors) reopened this Sep 2, 2016

Microsoft Contribution License Agreements (msftclas) added the cla-not-required label Sep 2, 2016

JamesWTruher reviewed Sep 2, 2016
View reviewed changes

Dongbo Wang (daxian-dbw) added Review - Waiting on Author and removed Review - Needed labels Sep 2, 2016

Travis Plunk (TravisEz13) added Review - Needed and removed Review - Waiting on Author labels Sep 2, 2016

Yanbing (bingbing8) added 9 commits September 2, 2016 18:43

Update based on PR feedback

40ad73b

updated more cases using testcase

0d8f3f9

update the format of the array

17a518b

Update to investigate an failure only happy in CI system

58486a7

update one test in OutErrorVariable

daf1bf7

update a test title

3895779

Fixed tests which has impact on other tests

71feca8

remove line to clean up the global variable since it has been changed…

ea5cb25

… o scipt scope.

Removed two lines for debugging purpuse

650c73a

sergei (vors) assigned James Truher (JamesWTruher) Sep 8, 2016

JamesWTruher reviewed Sep 9, 2016
View reviewed changes

Updated based on Jim's feedback.

ae43dae

vors reviewed Sep 12, 2016
View reviewed changes

sergei (vors) assigned sergei (vors) and unassigned James Truher (JamesWTruher) Sep 12, 2016

sergei (vors) added Review - Waiting on Author and removed Review - Needed labels Sep 12, 2016

Yanbing (bingbing8) added 2 commits September 12, 2016 13:40

remove file csharp.nuget.props from the PR. It was added by mistake.

4e54e84

added new lines at the end of the files.

08758ca

Travis Plunk (TravisEz13) added Review - Needed and removed Review - Waiting on Author labels Sep 12, 2016

sergei (vors) merged commit fd4552b into PowerShell:master Sep 13, 2016

Thatgfsj (Thatgfsj) pushed a commit to Thatgfsj-contribs/PowerShell that referenced this pull request Aug 6, 2026

Added tests for hashtabletoPSCustomObjectConversion, OutErrorVairable…

294ecbb

… tests (PowerShell#2160)

		@@ -0,0 +1,10 @@
		<?xml version="1.0" encoding="utf-8" standalone="no"?>

Conversation

Yanbing (bingbing8) commented Sep 1, 2016

Uh oh!

Microsoft Contribution License Agreements (msftclas) commented Sep 1, 2016

Uh oh!

Microsoft Contribution License Agreements (msftclas) commented Sep 2, 2016

Uh oh!

James Truher (JamesWTruher) commented Sep 2, 2016 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

James Truher (JamesWTruher) commented Sep 2, 2016

Uh oh!

James Truher (JamesWTruher) commented Sep 2, 2016

Uh oh!

James Truher (JamesWTruher) Sep 2, 2016

Choose a reason for hiding this comment

Uh oh!

Yanbing (bingbing8) Sep 3, 2016

Choose a reason for hiding this comment

Uh oh!

Yanbing (bingbing8) commented Sep 2, 2016

Uh oh!

Yanbing (bingbing8) commented Sep 2, 2016

Uh oh!

Yanbing (bingbing8) commented Sep 2, 2016

Uh oh!

Yanbing (bingbing8) commented Sep 7, 2016

Uh oh!

sergei (vors) commented Sep 8, 2016

Uh oh!

James Truher (JamesWTruher) Sep 9, 2016

Choose a reason for hiding this comment

Uh oh!

Yanbing (bingbing8) Sep 11, 2016

Choose a reason for hiding this comment

Uh oh!

James Truher (JamesWTruher) commented Sep 9, 2016 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Yanbing (bingbing8) commented Sep 12, 2016

Uh oh!

James Truher (JamesWTruher) commented Sep 12, 2016

Uh oh!

sergei (vors) Sep 12, 2016

Choose a reason for hiding this comment

Uh oh!

Yanbing (bingbing8) Sep 12, 2016

Choose a reason for hiding this comment

Uh oh!

sergei (vors) commented Sep 12, 2016

Uh oh!

Reviewers

Uh oh!

Assignees

Labels

Projects

Milestone

Development

Uh oh!

6 participants

James Truher (JamesWTruher) commented Sep 2, 2016 •

edited

Loading

James Truher (JamesWTruher) commented Sep 9, 2016 •

edited

Loading