Merge lp:~elopio/snappy/no_patch_defaultCacheDir into lp:~snappy-dev/snappy/snappy-moved-to-github

Proposed by Leo Arias
Status: Merged
Approved by: Michael Vogt
Approved revision: 509
Merged at revision: 509
Proposed branch: lp:~elopio/snappy/no_patch_defaultCacheDir
Merge into: lp:~snappy-dev/snappy/snappy-moved-to-github
Prerequisite: lp:~elopio/snappy/fix1464502-patch_defaultCacheDir
Diff against target: 27 lines (+0/-3)
1 file modified
partition/bootloader_uboot_test.go (+0/-3)
To merge this branch: bzr merge lp:~elopio/snappy/no_patch_defaultCacheDir
Reviewer Review Type Date Requested Status
Michael Vogt (community) Approve
Review via email: mp+261806@code.launchpad.net

Commit message

In the PartitionTestSuite, remove the unnecesary patches for defaultCacheDir.

Description of the change

After digging in bug #1464502, I found that there are some places where we patch the defaultCacheDir, but that var is not used as we are setting the hardwareSpecFile to a temp dir.

To post a comment you must log in.
Revision history for this message
Michael Vogt (mvo) wrote :

Thanks, nice catch. This part of the code got some overhaul recently in the lp:~mvo/snappy/snappy-improved-developer-mode[2-4] branches. Nice cleanup in any case!

Revision history for this message
Michael Vogt (mvo) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'partition/bootloader_uboot_test.go'
--- partition/bootloader_uboot_test.go 2015-06-12 05:27:29 +0000
+++ partition/bootloader_uboot_test.go 2015-06-12 05:27:29 +0000
@@ -204,7 +204,6 @@
204204
205 // mock the hardwareYaml and the cacheDir205 // mock the hardwareYaml and the cacheDir
206 p.hardwareSpecFile = makeHardwareYaml(c, "bootloader: grub")206 p.hardwareSpecFile = makeHardwareYaml(c, "bootloader: grub")
207 defaultCacheDir = c.MkDir()
208207
209 err = bootloader.HandleAssets()208 err = bootloader.HandleAssets()
210 c.Assert(err, NotNil)209 c.Assert(err, NotNil)
@@ -221,7 +220,6 @@
221bootloader: u-boot220bootloader: u-boot
222partition-layout: inplace221partition-layout: inplace
223`)222`)
224 defaultCacheDir = c.MkDir()
225223
226 err = bootloader.HandleAssets()224 err = bootloader.HandleAssets()
227 c.Assert(err, NotNil)225 c.Assert(err, NotNil)
@@ -247,7 +245,6 @@
247 p.hardwareSpecFile = makeHardwareYaml(c, `245 p.hardwareSpecFile = makeHardwareYaml(c, `
248bootloader u-boot246bootloader u-boot
249`)247`)
250 defaultCacheDir = c.MkDir()
251248
252 c.Assert(bootloader.HandleAssets(), NotNil)249 c.Assert(bootloader.HandleAssets(), NotNil)
253}250}

Subscribers

People subscribed via source and target branches