You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@couchdb.apache.org by am...@apache.org on 2019/09/24 18:59:39 UTC
[couchdb-fauxton] branch master updated: Remove deprecated size
properties from .info() (#1222)
This is an automated email from the ASF dual-hosted git repository.
amaranhao pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/couchdb-fauxton.git
The following commit(s) were added to refs/heads/master by this push:
new 9dada0e Remove deprecated size properties from .info() (#1222)
9dada0e is described below
commit 9dada0e73f33e3143e3fcab4a9e4f6486816f2c3
Author: Alexis Côté <po...@users.noreply.github.com>
AuthorDate: Tue Sep 24 14:59:34 2019 -0400
Remove deprecated size properties from .info() (#1222)
* Remove deprecated size properties from .info()
* Apply review comments
---
app/addons/databases/__tests__/components.test.js | 14 +++++++-------
app/addons/databases/__tests__/stores.test.js | 13 ++++++++-----
app/addons/databases/stores.js | 3 ++-
.../documents/designdocinfo/components/DesignDocInfo.js | 5 +++--
4 files changed, 20 insertions(+), 15 deletions(-)
diff --git a/app/addons/databases/__tests__/components.test.js b/app/addons/databases/__tests__/components.test.js
index c1f83a1..6522e5d 100644
--- a/app/addons/databases/__tests__/components.test.js
+++ b/app/addons/databases/__tests__/components.test.js
@@ -116,7 +116,7 @@ describe('DatabaseTable', () => {
beforeEach(() => {
Actions.updateDatabases({
dbList: ['db1'],
- databaseDetails: [{db_name: 'db1', doc_count: 0, doc_del_count: 0}],
+ databaseDetails: [{db_name: 'db1', doc_count: 0, doc_del_count: 0, sizes: {}}],
failedDbs: ['db1'],
fullDbList: ['db1']
});
@@ -165,7 +165,7 @@ describe('DatabaseTable', () => {
Actions.updateDatabases({
dbList: ['db1'],
- databaseDetails: [{db_name: 'db1', doc_count: 0, doc_del_count: 0}],
+ databaseDetails: [{db_name: 'db1', doc_count: 0, doc_del_count: 0, sizes: {}}],
failedDbs: [],
fullDbList: ['db1']
});
@@ -188,7 +188,7 @@ describe('DatabaseTable', () => {
it('shows error message if row marked as failed to load', () => {
Actions.updateDatabases({
dbList: ['db1'],
- databaseDetails: [{db_name: 'db1', doc_count: 0, doc_del_count: 0}],
+ databaseDetails: [{db_name: 'db1', doc_count: 0, doc_del_count: 0, sizes: {}}],
failedDbs: ['db1'],
fullDbList: ['db1']
});
@@ -204,7 +204,7 @@ describe('DatabaseTable', () => {
it('shows no error if row marked as loaded', () => {
Actions.updateDatabases({
dbList: ['db1'],
- databaseDetails: [{db_name: 'db1', doc_count: 0, doc_del_count: 0}],
+ databaseDetails: [{db_name: 'db1', doc_count: 0, doc_del_count: 0, sizes: {}}],
failedDbs: [],
fullDbList: ['db1']
});
@@ -221,7 +221,7 @@ describe('DatabaseTable', () => {
it('shows Partitioned column only when prop is set to true', () => {
Actions.updateDatabases({
dbList: ['db1'],
- databaseDetails: [{db_name: 'db1', doc_count: 0, doc_del_count: 0, props: {partitioned: true}}],
+ databaseDetails: [{db_name: 'db1', doc_count: 0, doc_del_count: 0, sizes:{}, props: {partitioned: true}}],
failedDbs: [],
fullDbList: ['db1']
});
@@ -245,8 +245,8 @@ describe('DatabaseTable', () => {
Actions.updateDatabases({
dbList: ['db1', 'db2'],
databaseDetails: [
- {db_name: 'db1', doc_count: 1, doc_del_count: 0, props: {partitioned: true}},
- {db_name: 'db2', doc_count: 2, doc_del_count: 0, props: {partitioned: false}}
+ {db_name: 'db1', doc_count: 1, doc_del_count: 0, sizes: {}, props: {partitioned: true}},
+ {db_name: 'db2', doc_count: 2, doc_del_count: 0, sizes: {}, props: {partitioned: false}}
],
failedDbs: [],
fullDbList: ['db1', 'db2']
diff --git a/app/addons/databases/__tests__/stores.test.js b/app/addons/databases/__tests__/stores.test.js
index d3483d4..0e94b11 100644
--- a/app/addons/databases/__tests__/stores.test.js
+++ b/app/addons/databases/__tests__/stores.test.js
@@ -27,7 +27,7 @@ describe('Databases Store', function () {
it('marks failed detail fetches as failed dbs', () => {
DatabaseActions.updateDatabases({
dbList: ['db1', 'db2'],
- databaseDetails: [{db_name: 'db1'}, {db_name: 'db2'}],
+ databaseDetails: [{db_name: 'db1', sizes: {}}, {db_name: 'db2', sizes: {}}],
failedDbs: ['db1']
});
@@ -39,7 +39,7 @@ describe('Databases Store', function () {
it('unions details', () => {
DatabaseActions.updateDatabases({
dbList: ['db1'],
- databaseDetails: [{db_name: 'db1', doc_count: 5, doc_del_count: 3}],
+ databaseDetails: [{db_name: 'db1', doc_count: 5, doc_del_count: 3, sizes: {}}],
failedDbs: []
});
@@ -60,15 +60,18 @@ describe('Databases Store', function () {
expect(!store.doesDatabaseExist('db3')).toBeTruthy();
});
- it('uses the data_size prop', () => {
+ it('uses the sizes.active prop', () => {
DatabaseActions.updateDatabases({
dbList: ['db1'],
databaseDetails: [{
db_name: 'db1',
doc_count: 5,
doc_del_count: 3,
- data_size: 1337,
- disk_size: 0
+ sizes: {
+ active: 1337,
+ external: 0,
+ file: 0,
+ }
}],
failedDbs: []
});
diff --git a/app/addons/databases/stores.js b/app/addons/databases/stores.js
index 2f77c4d..b3e05bc 100644
--- a/app/addons/databases/stores.js
+++ b/app/addons/databases/stores.js
@@ -112,7 +112,8 @@ const DatabasesStoreConstructor = FauxtonAPI.Store.extend({
if (!details) {
return {};
}
- const dataSize = details.data_size || details.disk_size || 0;
+ const {sizes} = details;
+ const dataSize = sizes.active || 0;
return {
dataSize: Helpers.formatSize(dataSize),
diff --git a/app/addons/documents/designdocinfo/components/DesignDocInfo.js b/app/addons/documents/designdocinfo/components/DesignDocInfo.js
index 70d6050..3c80c94 100644
--- a/app/addons/documents/designdocinfo/components/DesignDocInfo.js
+++ b/app/addons/documents/designdocinfo/components/DesignDocInfo.js
@@ -56,8 +56,9 @@ export default class DesignDocInfo extends React.Component {
return <LoadLines />;
}
const viewIndex = this.props.viewIndex;
- const actualSize = (viewIndex.data_size) ? viewIndex.data_size.toLocaleString('en') : 0;
- const dataSize = (viewIndex.disk_size) ? viewIndex.disk_size.toLocaleString('en') : 0;
+ const {sizes} = viewIndex;
+ const actualSize = (sizes.active) ? sizes.active.toLocaleString('en') : 0;
+ const dataSize = (sizes.external) ? sizes.external.toLocaleString('en') : 0;
return (
<div className="metadata-page">