[Buildroot] [PATCH 2/3] testing/tests/package/test_python: refactor to support better code reuse
thomas.petazzoni at free-electrons.com
Thu Jul 6 07:08:21 UTC 2017
On Thu, 6 Jul 2017 00:09:10 +0200, Arnout Vandecappelle wrote:
> >>> + def login(self, timeout=5):
> >>> cpio_file = os.path.join(self.builddir, "images", "rootfs.cpio")
> >>> self.emulator.boot(arch="armv5",
> >>> kernel="builtin",
> >>> - options=["-initrd", cpio_file])
> >>> + options=["-initrd", cpio_file],
> >>> + timeout)
> >>> self.emulator.login()
> >>> - cmd = "python --version 2>&1 | grep '^Python 2'"
> >>> - _, exit_code = self.emulator.run(cmd)
> >>> - self.assertEqual(exit_code, 0)
> >> I'm not a big fan of having a method called .login() also start the
> >> interpreter, it feels a bit weird.
> > I'll split it in the code into custom "boot()" and "login()" in v2,
> > unless you have something else in mind.
> I *think* this is a mistake on Thomas's part, that he thought the login()
> function would boot, login and start Python. I.e., he missed the - in front of
> the line. For me at least it's very natural that the login() function does the
> boot + login.
Ah, indeed. My bad. It's perfectly fine for me if a .login() or .boot()
method does both the boot+login.
Sorry for having misread the patch :/
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux and Kernel engineering
More information about the buildroot