Skip to content

Commit 2cac180

Browse files
xavierfigueroavpombredanne
authored andcommitted
Resolve merge conflicts on tests
This is a following the merge of PR #110 for ticket #91 Signed-off-by: Philippe Ombredanne <pombredanne@nexb.com>
2 parents a3ab3f4 + 4fb130a commit 2cac180

File tree

6 files changed

+667
-33
lines changed

6 files changed

+667
-33
lines changed

data/SPDXRdfExample.rdf

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -118,7 +118,6 @@
118118
<referencesFile>
119119
<File rdf:about="https://spdx.org/spdxdocs/spdx-example-444504E0-4F89-41D3-9A0C-0305E82C3301#SPDXRef-File2">
120120
<copyrightText>Copyright 2010, 2011 Source Auditor Inc.</copyrightText>
121-
<licenseComments></licenseComments>
122121
<licenseInfoInFile rdf:resource="http://spdx.org/licenses/Apache-2.0"/>
123122
<licenseConcluded rdf:resource="http://spdx.org/licenses/Apache-2.0"/>
124123
<fileType rdf:resource="http://spdx.org/rdf/terms#fileType_source"/>

setup.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -28,6 +28,7 @@ def test_suite():
2828
'ply',
2929
'rdflib',
3030
'six',
31+
'pyyaml',
3132
],
3233
entry_points={
3334
'console_scripts': [

spdx/parsers/rdf.py

Lines changed: 13 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -111,7 +111,7 @@ def to_special_value(self, value):
111111
elif value == self.spdx_namespace.unknown:
112112
return utils.UnKnown()
113113
else:
114-
return value
114+
return six.text_type(value)
115115

116116

117117
class LicenseParser(BaseParser):
@@ -140,7 +140,7 @@ def handle_lics(self, lics):
140140
if special == lics:
141141
if self.LICS_REF_REGEX.match(lics):
142142
# Is a license ref i.e LicenseRef-1
143-
return document.License.from_identifier(lics)
143+
return document.License.from_identifier(six.text_type(lics))
144144
else:
145145
# Not a known license form
146146
raise SPDXValueError('License')
@@ -169,7 +169,7 @@ def get_extr_license_ident(self, extr_lic):
169169

170170
identifier_tripple = identifier_tripples[0]
171171
_s, _p, identifier = identifier_tripple
172-
return identifier
172+
return six.text_type(identifier)
173173

174174
def get_extr_license_text(self, extr_lic):
175175
"""
@@ -188,7 +188,7 @@ def get_extr_license_text(self, extr_lic):
188188

189189
text_tripple = text_tripples[0]
190190
_s, _p, text = text_tripple
191-
return text
191+
return six.text_type(text)
192192

193193
def get_extr_lic_name(self, extr_lic):
194194
"""
@@ -200,14 +200,14 @@ def get_extr_lic_name(self, extr_lic):
200200
return
201201
elif len(extr_name_list) == 0:
202202
return
203-
return self.to_special_value(extr_name_list[0][2])
203+
return six.text_type(self.to_special_value(extr_name_list[0][2]))
204204

205205
def get_extr_lics_xref(self, extr_lic):
206206
"""
207207
Return a list of cross references.
208208
"""
209209
xrefs = list(self.graph.triples((extr_lic, RDFS.seeAlso, None)))
210-
return map(lambda xref_triple: xref_triple[2], xrefs)
210+
return map(lambda xref_triple: six.text_type(xref_triple[2]), xrefs)
211211

212212
def get_extr_lics_comment(self, extr_lics):
213213
"""
@@ -219,7 +219,7 @@ def get_extr_lics_comment(self, extr_lics):
219219
self.more_than_one_error('extracted license comment')
220220
return
221221
elif len(comment_list) == 1:
222-
return comment_list[0][2]
222+
return six.text_type(comment_list[0][2])
223223
else:
224224
return
225225

@@ -249,7 +249,7 @@ def parse_only_extr_license(self, extr_lic):
249249
lic.full_name = name
250250
if comment is not None:
251251
lic.comment = comment
252-
lic.cross_ref = map(lambda x: six.text_type(x), xrefs)
252+
lic.cross_ref = xrefs
253253
return lic
254254

255255
def handle_extracted_license(self, extr_lic):
@@ -352,7 +352,7 @@ def parse_package(self, p_term):
352352
def p_pkg_cr_text(self, p_term, predicate):
353353
try:
354354
for _, _, text in self.graph.triples((p_term, predicate, None)):
355-
self.builder.set_pkg_cr_text(self.doc, self.to_special_value(six.text_type(text)))
355+
self.builder.set_pkg_cr_text(self.doc, six.text_type(self.to_special_value(text)))
356356
except CardinalityError:
357357
self.more_than_one_error('package copyright text')
358358

@@ -653,7 +653,7 @@ def p_file_lic_info(self, f_term, predicate):
653653
def p_file_spdx_id(self, f_term, predicate):
654654
try:
655655
try:
656-
self.builder.set_file_spdx_id(self.doc, f_term)
656+
self.builder.set_file_spdx_id(self.doc, six.text_type(f_term))
657657
except SPDXValueError:
658658
self.value_error('FILE_SPDX_ID_VALUE', f_term)
659659
except CardinalityError:
@@ -878,7 +878,7 @@ def parse_annotation(self, r_term):
878878
annotation_type = self.get_annotation_type(r_term)
879879
self.builder.add_annotation_type(self.doc, annotation_type)
880880
try:
881-
self.builder.set_annotation_spdx_id(self.doc, r_term)
881+
self.builder.set_annotation_spdx_id(self.doc, six.text_type(r_term))
882882
except CardinalityError:
883883
self.more_than_one_error('SPDX Identifier Reference')
884884

@@ -888,7 +888,7 @@ def get_annotation_type(self, r_term):
888888
for _, _, typ in self.graph.triples((
889889
r_term, self.spdx_namespace['annotationType'], None)):
890890
if typ is not None:
891-
return typ
891+
return six.text_type(typ)
892892
else:
893893
self.error = True
894894
msg = 'Annotation must have exactly one annotation type.'
@@ -1034,7 +1034,7 @@ def parse_doc_fields(self, doc_term):
10341034
"""Parses the version, data license, name, SPDX Identifier, namespace,
10351035
and comment."""
10361036
try:
1037-
self.builder.set_doc_spdx_id(self.doc, doc_term)
1037+
self.builder.set_doc_spdx_id(self.doc, six.text_type(doc_term))
10381038
except SPDXValueError:
10391039
self.value_error('DOC_SPDX_ID_VALUE', doc_term)
10401040
try:

0 commit comments

Comments
 (0)