1146095 - StorageTests are disabled and broken

Status: RESOLVED FIXED

(Firefox for iOS :: Build & Test, defect)

Description

[Sachin Irukula]

User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_10_2) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/41.0.2272.101 Safari/537.36

Steps to reproduce:

1 . Checkout a the trunk
2. Follow the instructions to build.


Actual results:

When I try to build the Client an error Missing argument for parameter 'rootPath' in call is throw in some of the files 
firefox-ios/StorageTests/SQLiteRemoteClientsAndTabsTests.swift
firefox-ios/StorageTests/TestTableTable.swift etc.

I think the problem is when MockFiles is instantiated it requires a rootPath parameter which is not being passed currently. MockFiles inherits from FileAccessor whose initializer takes rootPath as parameter.


Expected results:

Build should have been a success.

Comment 1

[Sachin Irukula]

I did some more digging and found out that this started happening after the commit https://github.com/mozilla/firefox-ios/commit/82fb582cd79666bd866b14de60766d1cf481f4c9

In this commit init method has been added for superclass FileAccessor and remove method parameters have been changed and similar changes should happen in the subclass MockFiles which didn't.

Comment 2

[Brian Nicholson (:bnicholson)]

You probably need to clear your DerivedData -- StorageTests aren't running at all right now, so they shouldn't be failing in a working build.

That said, StorageTests shouldn't be disabled in the first place. StorageTests were part of the Storage framework, which was deleted in bug 1145335. Looks like we forgot to move them into the Client scheme.

Comment 3

[Brian Nicholson (:bnicholson)]

Oops, didn't mean to close this yet.

Comment 4

[Brian Nicholson (:bnicholson)]

Attachment: Pull request

Comment 5

[Nick Alexander :nalexander [he/him]]

Comment on attachment 8581717 [details]
Pull request

sachin: thanks for the report and the sleuthing!  This was all my fault.  bnicholson, patch looks good and tests pass on simulator locally.  (And ClientTests does include StorageTests now.)