test: update test names to better reflect the tests assertions

Author: erezrokah

lib/package/sns/validateSnsServiceProxy.test.js

@@ -25,7 +25,7 @@ describe('#validateSnsServiceProxy()', () => {
     serverlessApigatewayServiceProxy = new ServerlessApigatewayServiceProxy(serverless, options)
   })
 
-  it('should validate the "topicName" property existence', async () => {
+  it('should throw error if the "topicName" property doesn\'t exist', async () => {
     serverlessApigatewayServiceProxy.validated = {
       events: [
         {
@@ -43,7 +43,7 @@ describe('#validateSnsServiceProxy()', () => {
     )
   })
 
-  it('should validate the "topicName" property if it is string or AWS intrinsic function', async () => {
+  it('should throw error if the "topicName" property is not a string or an AWS intrinsic function', async () => {
     serverlessApigatewayServiceProxy.validated = {
       events: [
         {
@@ -62,7 +62,7 @@ describe('#validateSnsServiceProxy()', () => {
     )
   })
 
-  it('should validate the "topicName" property that AWS intrinsic function is correct syntax', async () => {
+  it('should throw error if the "topicName" property is missing the AWS intrinsic function "Fn::GetAtt"', async () => {
     serverlessApigatewayServiceProxy.validated = {
       events: [
         {
@@ -81,7 +81,7 @@ describe('#validateSnsServiceProxy()', () => {
     )
   })
 
-  it('should validate the "topicName" property if wrong AWS intrinsic function is specified', async () => {
+  it('should throw error if the "topicName" property is an intrinsic function "Fn::GetAtt" but specifies a property other than TopicName', async () => {
     serverlessApigatewayServiceProxy.validated = {
       events: [
         {