Skip to content
This repository has been archived by the owner on Dec 11, 2019. It is now read-only.

Commit

Permalink
Cleaning up leveldown references
Browse files Browse the repository at this point in the history
- commented out code from ledger
- stubs used in unit testing
- electron builder hack (no longer needed)

Auditors: @diracdeltas
  • Loading branch information
bsclifton committed Jun 29, 2017
1 parent 643996d commit 70bfd75
Show file tree
Hide file tree
Showing 5 changed files with 0 additions and 41 deletions.
29 changes: 0 additions & 29 deletions app/ledger.js
Original file line number Diff line number Diff line change
Expand Up @@ -1862,35 +1862,6 @@ var run = (delayTime) => {
console.log('what? wait, how can this happen?')
}

/*
* ledger client utilities
*/

/* code that may never be needed...
var rulesV2Reset = (callback) => {
if (clientOptions.verboseP) console.log('\n\nreset rulesets')
if (!v2RulesetDB) return
if (client) {
delete client.state.rulesV2Stamp
client.state.updatesStamp = underscore.now()
}
v2RulesetDB.close((err) => {
if (err) console.log(v2RulesetPath + ' close error: ' + err.toString())
v2RulesetDB = null
require('leveldown').destroy(pathName(v2RulesetPath), (err) => {
if (err) console.log(v2RulesetPath + ' destroy error: ' + err.toString())
v2RulesetDB = levelup(pathName(v2RulesetPath))
callback()
})
})
}
*/

var getStateInfo = (state) => {
var ballots, i, transaction
var info = state.paymentInfo
Expand Down
1 change: 0 additions & 1 deletion test/unit/app/browser/reducers/tabsReducerTest.js
Original file line number Diff line number Diff line change
Expand Up @@ -89,7 +89,6 @@ describe('tabsReducer unit tests', function () {
})
mockery.registerMock('electron', fakeElectron)
mockery.registerMock('ad-block', fakeAdBlock)
mockery.registerMock('leveldown', {})

this.tabsAPI = {
isDevToolsFocused: (tabId) => {
Expand Down
1 change: 0 additions & 1 deletion test/unit/app/browser/tabsTest.js
Original file line number Diff line number Diff line change
Expand Up @@ -65,7 +65,6 @@ describe('tabs API unit tests', function () {

mockery.registerMock('electron', fakeElectron)
mockery.registerMock('ad-block', fakeAdBlock)
mockery.registerMock('leveldown', {})
mockery.registerMock('../../js/stores/appStore', this.appStore)
mockery.registerMock('../filtering', {
isResourceEnabled: (resourceName, url, isPrivate) => false
Expand Down
9 changes: 0 additions & 9 deletions tools/electronBuilderHack.js
Original file line number Diff line number Diff line change
Expand Up @@ -15,15 +15,6 @@ const rebuildCmd = '"../.bin/node-gyp" rebuild'
const isDarwin = process.platform === 'darwin'

const cmds = []
if (process.env.TARGET_ARCH === 'ia32') {
cmds.push(
'cp .npmrc ./node_modules/leveldown',
'cd ./node_modules/leveldown',
// leveldown prebuild isn't respecting the npm_config_arch env
// for some reason, so always compile it.
'npm run rebuild'
)
}

if (isDarwin) {
cmds.push(
Expand Down
1 change: 0 additions & 1 deletion tools/lib/ignoredPaths.js
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,6 @@ module.exports = [
'Release/obj',
'obj.target',
'ad-block/(node_modules|test|perf|sample|scripts|test|vendor|ABPFilterParserData.dat)',
'leveldown/(node_modules|src|iojs)',
'tracking-protection/(node_modules|test|data|scripts|vendor)',
'tracking-protection/build/node_modules',
'tracking-protection/build/Release/(sample.exe|test.exe)',
Expand Down

0 comments on commit 70bfd75

Please sign in to comment.