Fix action translations edge cases

ui/app/components/transaction-action/transaction-action.component.js

@@ -31,7 +31,7 @@ export default class TransactionAction extends PureComponent {
     const { transactionAction } = this.state
     const { transaction, methodData } = this.props
     const { data, done } = methodData
-    const { name } = data
+    const { name = '' } = data
 
     if (!done || transactionAction) {
       return

ui/app/helpers/common.util.js

@@ -1,4 +1,4 @@
-export function camelCaseToCapitalize (str) {
+export function camelCaseToCapitalize (str = '') {
   return str
     .replace(/([A-Z])/g, ' $1')
     .replace(/^./, str => str.toUpperCase())

ui/app/helpers/tests/common.util.test.js

@@ -5,6 +5,10 @@ describe('Common utils', () => {
   describe('camelCaseToCapitalize', () => {
     it('should return a capitalized string from a camel-cased string', () => {
       const tests = [
+        {
+          test: undefined,
+          expected: '',
+        },
         {
           test: '',
           expected: '',

ui/app/helpers/transactions.util.js

@@ -44,6 +44,12 @@ export function isConfirmDeployContract (txData = {}) {
   return !txParams.to
 }
 
+/**
+ * Returns the action of a transaction as a key to be passed into the translator.
+ * @param {Object} transaction - txData object
+ * @param {Object} methodData - Data returned from eth-method-registry
+ * @returns {string|undefined}
+ */
 export async function getTransactionActionKey (transaction, methodData) {
   const { txParams: { data, to } = {}, msgParams, type } = transaction
 
@@ -80,6 +86,8 @@ export async function getTransactionActionKey (transaction, methodData) {
         return APPROVE_ACTION_KEY
       case TOKEN_METHOD_TRANSFER_FROM:
         return TRANSFER_FROM_ACTION_KEY
+      default:
+        return undefined
     }
   } else {
     return SEND_ETHER_ACTION_KEY