Rewrite TrackerAttributesTest to parse RDFa [#1886102]

Similarly to #1867096: Rewrite RdfaMarkupTest to parse RDFa, we should also update TrackerAttributesTest to use easyrdf to parse the RDFa output and verify that the output is what we expect.

This issue depends on the easyrdf library: #1866858: Test RDFa by parsing RDFa (add the easyrdf library).

Comment	File	Size	Author
#6	1886102_TrackerAttributesTest_6.patch	10.58 KB	scor
#6
#6	interdiff.txt	2.09 KB	scor
#4	1886102_TrackerAttributesTest_4.patch	10.24 KB	scor
#4
#4	interdiff.txt	3.34 KB	scor
#2	1886102_TrackerAttributesTest_2.patch	8.35 KB	scor
#2
#1	1886102_TrackerAttributesTest_1.patch	1.51 KB	scor
#1

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

scor CreditAttribution: scor commented 11 January 2013 at 13:42

Status:

Active

» Needs work

File	Size
1886102_TrackerAttributesTest_1.patch	1.51 KB

Here is a start (the easyrdf library is not included in this patch).

Comment #2

scor CreditAttribution: scor commented 14 January 2013 at 19:37

Status:

Needs work

» Needs review

File	Size
1886102_TrackerAttributesTest_2.patch	8.35 KB

this should cover all the tests. still using the standard profile at the moment.

Comment #3

Anonymous (not verified) CreditAttribution: Anonymous commented 15 January 2013 at 14:57

Status:

Needs review

» Needs work

We can remove the dependence on standard profile by:

Removing 'rdf_test', which isn't actually necessary for the test.
Adding a call to drupalCreateContentType to create the article content type, like AggregatorTestBase does.

EDIT: actually, removing rdf_test probably isn't necessary, but should be done anyway since it isn't really required.

Comment #4

scor CreditAttribution: scor commented 17 January 2013 at 17:03

Status:

Needs work

» Needs review

File	Size
interdiff.txt	3.34 KB
1886102_TrackerAttributesTest_4.patch	10.24 KB

- removed dependency on standard profile by creating the article content type in setUp()
- removed rdf_test dependency
- moved base_uri in setUp()

Comment #5

scor CreditAttribution: scor commented 17 January 2013 at 17:06

oops, forgot to fix the 'definition of' -> 'Contains' at the beginning as well. will wait for more comment to roll a new patch. keep this needs review for now.

Comment #6

scor CreditAttribution: scor commented 17 January 2013 at 22:30

File	Size
interdiff.txt	2.09 KB
1886102_TrackerAttributesTest_6.patch	10.58 KB

also improved the code to fetch last activity date from the db.

Comment #7

Anonymous (not verified) CreditAttribution: Anonymous commented 18 January 2013 at 13:50

Status:

Needs review

» Reviewed & tested by the community

Looks good, thanks!

Like the comments test update which was committed yesterday, this removes the dependency on standard profile and makes the tests more accurate by actually parsing RDFa.