Closed Bug 1313467 Opened 3 years ago Closed 3 years ago

Convert XPCOM test TestAutoPtr to a gtest

Categories

(Core :: XPCOM, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla52
Tracking Status
firefox52 --- fixed

People

(Reporter: erahm, Assigned: erahm)

References

Details

Attachments

(4 files)

Convert xpcom/tests/TestAutoPtr.cpp to a gtest and move to xpcom/tests/gtest/.
This is a test for nsAutoPtr, we shouldn't be testing RefPtr here.

MozReview-Commit-ID: IqwgINAuYar
Attachment #8805366 - Flags: review?(nfroyd)
Assignee: nobody → erahm
Status: NEW → ASSIGNED
This just moves code into separate functions. No logic changes were made.

MozReview-Commit-ID: 98NqhCb4T67
Attachment #8805367 - Flags: review?(nfroyd)
MozReview-Commit-ID: AfMDPcBjf3Y
Attachment #8805368 - Flags: review?(nfroyd)
MozReview-Commit-ID: EOKHAQyZ4fZ
Attachment #8805369 - Flags: review?(nfroyd)
Attachment #8805366 - Flags: review?(nfroyd) → review+
Attachment #8805367 - Flags: review?(nfroyd) → review+
Attachment #8805368 - Flags: review?(nfroyd) → review+
Comment on attachment 8805369 [details] [diff] [review]
Part 4: Convert TestAutoPtr to a gtest

Review of attachment 8805369 [details] [diff] [review]:
-----------------------------------------------------------------

r=me with changes below.

::: xpcom/tests/gtest/TestAutoPtr.cpp
@@ +7,5 @@
>  #include "nsAutoPtr.h"
> +#include "gtest/gtest.h"
> +//#include <stdio.h>
> +//#include "nscore.h"
> +//#include "mozilla/Attributes.h"

Might as well just delete these while you're at it, rather than commenting them out?

::: xpcom/tests/moz.build
@@ -45,5 @@
>  
>  XPCSHELL_TESTS_MANIFESTS += ['unit/xpcshell.ini']
>  
>  GeckoCppUnitTests([
> -    'TestAutoPtr',

You need to remove this from testing/cppunittest.ini as well.
Attachment #8805369 - Flags: review?(nfroyd) → review+
You need to log in before you can comment on or make changes to this bug.