<feed xmlns='http://www.w3.org/2005/Atom'>
<title>delta/buildstream.git/tests/frontend/source_checkout.py, branch bschubert/python38</title>
<subtitle>gitlab.com: buildstream/buildstream.git
</subtitle>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/buildstream.git/'/>
<entry>
<title>Reformat code using Black</title>
<updated>2019-11-14T21:21:06+00:00</updated>
<author>
<name>Chandan Singh</name>
<email>csingh43@bloomberg.net</email>
</author>
<published>2019-11-11T17:07:09+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/buildstream.git/commit/?id=122177153b14664a0e4fed85aa4f22b87cfabf56'/>
<id>122177153b14664a0e4fed85aa4f22b87cfabf56</id>
<content type='text'>
As discussed over the mailing list, reformat code using Black. This is a
one-off change to reformat all our codebase. Moving forward, we
shouldn't expect such blanket reformats. Rather, we expect each change
to already comply with the Black formatting style.
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
As discussed over the mailing list, reformat code using Black. This is a
one-off change to reformat all our codebase. Moving forward, we
shouldn't expect such blanket reformats. Rather, we expect each change
to already comply with the Black formatting style.
</pre>
</div>
</content>
</entry>
<entry>
<title>source_checkout.py: Add --compression tests for source checkout</title>
<updated>2019-09-12T15:29:13+00:00</updated>
<author>
<name>James Ennis</name>
<email>james.ennis@codethink.co.uk</email>
</author>
<published>2019-09-09T11:40:02+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/buildstream.git/commit/?id=f047495827f38b38434e4caf0872aa9582d89046'/>
<id>f047495827f38b38434e4caf0872aa9582d89046</id>
<content type='text'>
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
</pre>
</div>
</content>
</entry>
<entry>
<title>source_checkout.py: Add test for --tar/--directory conflict</title>
<updated>2019-09-12T15:29:13+00:00</updated>
<author>
<name>James Ennis</name>
<email>james.ennis@codethink.co.uk</email>
</author>
<published>2019-09-09T09:05:11+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/buildstream.git/commit/?id=c8a06a4515e67e5007fc2f5cb3f83e3a775073c9'/>
<id>c8a06a4515e67e5007fc2f5cb3f83e3a775073c9</id>
<content type='text'>
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
</pre>
</div>
</content>
</entry>
<entry>
<title>cli.py: Make source checkout more consistent with artifact checkout</title>
<updated>2019-09-12T15:29:12+00:00</updated>
<author>
<name>James Ennis</name>
<email>james.ennis@codethink.co.uk</email>
</author>
<published>2019-09-06T16:53:54+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/buildstream.git/commit/?id=6cb668ade39dfe1cf5e23177b022492e1632b035'/>
<id>6cb668ade39dfe1cf5e23177b022492e1632b035</id>
<content type='text'>
This MR changes the behaviour of source checkout so that it is more
like artifact checkout. That is, a --directory option can be provided,
rather than a mandatory LOCATION.

In addition to this, the --tar option is no longer a boolean flag,
rather it expects a tarfile name (just like artifact checkout)

The appropriate tests have also been altered so that they support
the new API
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
This MR changes the behaviour of source checkout so that it is more
like artifact checkout. That is, a --directory option can be provided,
rather than a mandatory LOCATION.

In addition to this, the --tar option is no longer a boolean flag,
rather it expects a tarfile name (just like artifact checkout)

The appropriate tests have also been altered so that they support
the new API
</pre>
</div>
</content>
</entry>
<entry>
<title>source_checkout.py: Remove commented out touch() and add comment</title>
<updated>2019-09-12T15:29:12+00:00</updated>
<author>
<name>James Ennis</name>
<email>james.ennis@codethink.co.uk</email>
</author>
<published>2019-09-06T13:52:07+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/buildstream.git/commit/?id=344c3e4cd4934be12ddb0e44cfbb617114dd14dd'/>
<id>344c3e4cd4934be12ddb0e44cfbb617114dd14dd</id>
<content type='text'>
In "test_source_checkout_force()" we had a commented out line
of code. This has been removed and a more useful comment has been
added.
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
In "test_source_checkout_force()" we had a commented out line
of code. This has been removed and a more useful comment has been
added.
</pre>
</div>
</content>
</entry>
<entry>
<title>tests: Change all calls to _yaml.dump to _yaml.rountrip_dump</title>
<updated>2019-07-15T14:14:03+00:00</updated>
<author>
<name>Benjamin Schubert</name>
<email>ben.c.schubert@gmail.com</email>
</author>
<published>2019-06-28T16:01:23+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/buildstream.git/commit/?id=f7b461220ec2f1af3966ace12c21334f1f3b6457'/>
<id>f7b461220ec2f1af3966ace12c21334f1f3b6457</id>
<content type='text'>
Now that both are equivalent, we can skip the sanitization part before
the yaml call.
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Now that both are equivalent, we can skip the sanitization part before
the yaml call.
</pre>
</div>
</content>
</entry>
<entry>
<title>Fetch sources as needed for bst source checkout</title>
<updated>2019-06-27T08:46:27+00:00</updated>
<author>
<name>Jürg Billeter</name>
<email>j@bitron.ch</email>
</author>
<published>2019-06-26T13:59:21+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/buildstream.git/commit/?id=868ff50c721164a9c24e076e163bed02145c4624'/>
<id>868ff50c721164a9c24e076e163bed02145c4624</id>
<content type='text'>
This removes the --fetch option of bst source checkout, always enabling
fetch support. This is in line with fetching subprojects as needed.
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
This removes the --fetch option of bst source checkout, always enabling
fetch support. This is in line with fetching subprojects as needed.
</pre>
</div>
</content>
</entry>
<entry>
<title>plugintestutils: Rename 'plugintestutils' package to 'testing'</title>
<updated>2019-04-16T13:25:40+00:00</updated>
<author>
<name>Phil Dawson</name>
<email>phil.dawson@codethink.co.uk</email>
</author>
<published>2019-04-15T10:02:33+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/buildstream.git/commit/?id=97bbedbef15fc9edea5b04f66a30ec788da7f130'/>
<id>97bbedbef15fc9edea5b04f66a30ec788da7f130</id>
<content type='text'>
- Rename plugintestutils to testing.

- Don't run the tests from bst-plugins-template. This imports
  buildstream.plugintestutils so will have to be disabled to get
  through CI. This can be re nabled once bst-plugins-template has been
  patched.
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
- Rename plugintestutils to testing.

- Don't run the tests from bst-plugins-template. This imports
  buildstream.plugintestutils so will have to be disabled to get
  through CI. This can be re nabled once bst-plugins-template has been
  patched.
</pre>
</div>
</content>
</entry>
<entry>
<title>tests: str(datafiles) instead of a longer thing</title>
<updated>2019-03-21T14:57:31+00:00</updated>
<author>
<name>Angelos Evripiotis</name>
<email>jevripiotis@bloomberg.net</email>
</author>
<published>2019-03-20T11:06:42+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/buildstream.git/commit/?id=f2b15d86debd32241228f7b728016ed0057cd0d2'/>
<id>f2b15d86debd32241228f7b728016ed0057cd0d2</id>
<content type='text'>
Replace some popular copypasta.

This important-looking invocation:

    os.path.join(datafiles.dirname, datafiles.basename)

is equivalent to this shorter invocation:

    project = str(datafiles)

It seems like it's very popular copypasta, replace it with the shorter
one thus:

    # Use 'gsed' or 'sed' etc. as appropriate for your system.
    git config --global alias.sub '!f() { git grep --name-only --null "$1" | gxargs --null gsed --in-place --expression "s/$1/$2/g" ; }; f'
    git sub 'os.path.join(datafiles.dirname, datafiles.basename)' 'str(datafiles)'
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Replace some popular copypasta.

This important-looking invocation:

    os.path.join(datafiles.dirname, datafiles.basename)

is equivalent to this shorter invocation:

    project = str(datafiles)

It seems like it's very popular copypasta, replace it with the shorter
one thus:

    # Use 'gsed' or 'sed' etc. as appropriate for your system.
    git config --global alias.sub '!f() { git grep --name-only --null "$1" | gxargs --null gsed --in-place --expression "s/$1/$2/g" ; }; f'
    git sub 'os.path.join(datafiles.dirname, datafiles.basename)' 'str(datafiles)'
</pre>
</div>
</content>
</entry>
<entry>
<title>tests:lint: disable 'unused-import' checks on pytest fixtures</title>
<updated>2019-03-20T09:38:42+00:00</updated>
<author>
<name>Benjamin Schubert</name>
<email>ben.c.schubert@gmail.com</email>
</author>
<published>2019-03-19T18:36:38+00:00</published>
<link rel='alternate' type='text/html' href='http://git.baserock.org/cgit/delta/buildstream.git/commit/?id=da439e6658049bb79ca471f4eeac401e70c548f1'/>
<id>da439e6658049bb79ca471f4eeac401e70c548f1</id>
<content type='text'>
Pylint can't know that pytest's fixtures are used in a file and
therefore reports false positives. Silencing all those errors
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Pylint can't know that pytest's fixtures are used in a file and
therefore reports false positives. Silencing all those errors
</pre>
</div>
</content>
</entry>
</feed>
